Re: `save-excursion' defeated by `set-buffer'

[David Kastrup]

Stefan Monnier <monnier@iro.umontreal.ca> writes:

>> pointing to other ways of delivering a task doesn't prove
>
>> save-excursion+set-buffer
>
>> is wrong.
>
> There is no proof that save-excursion+set-buffer is wrong, because it
> just is not wrong: it's just a combination of function that performs
> something.  That's why the byte-compiler emits a warning and not an
> error message.

[...]

> So the warning was introduced to help Elisp coders improve their code.
> That's what byte-compiler warnings are for.

It fails to do so.  It does not _help_ Elisp coders improve their code
but rather _confuses_ them into improving their code.

The warning message would be more accurately written as

     "Warning: the maintainer general has detected that
               save-excursion+set-buffer may seriously hamper your code."

That's much more likely to be accurate than the current warning.

-- 
David Kastrup