Pierre Rouleau

Results 68 comments of Pierre Rouleau
trafficstars

I second what was said about this pull-request. Seems like nothing has changed on this repo since April 2016. I have forked the repo and have integrated the changes requested...

@Trevoke Here's my understanding: There are 2 commands: - ``(command-log-mode &optional ARG)`` toggles the command logging for the *current* buffer. Nothing else. - ``(global-command-log-mode &optional ARG)`` toggles the command logging...

Hi @stardiviner, I have forked this project and integrated your suggestion in the PR in [my fork of the project](https://github.com/pierre-rouleau/command-log-mode). I'd like to see the changes integrated here but there...

I was just reading the code. I tried setting a buffer with `read-only-mode` and applied `sp-add-to-previous-sexp` and it did refuse to execute with the error stating the buffer was read-only...

The code in emacs/src/callint.c that defines the behaviour checks buffer state when '*' is specified and returns an error if the buffer is read-only. IMO using the '\*' explicitly is...

It's possible that the *current* code has no side effect. But a language feature like that is a kind of defensive-programming mechanism. It does not hurt not having it in...

@ThibautVerron , the reason non-interactive functions do not have checks for read-only buffers might be performance. However a large chunck of code is called via interactive functions. If all these...

@ Fuco1, I modified the ticket title to properly describe what I reported. I am advocating using this defensive-programming utility that prevents interactive function to operate on a read-only buffer....

@ThibautVerron You are correct: an interactive spec is only checked for the function called interactively, either from the user action or from a `call-interactively` call, and not from a direct...

Erlang has lists that have comma separators enclosed in `[ ]` pairs, with each element separated by comma. It also has a statement terminator period. The following 2 lines are...