[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: `save-excursion' defeated by `set-buffer'
From: |
martin rudalics |
Subject: |
Re: `save-excursion' defeated by `set-buffer' |
Date: |
Mon, 11 Jan 2010 10:21:50 +0100 |
User-agent: |
Thunderbird 2.0.0.21 (Windows/20090302) |
> If you say that you find most (save-excursion (set-buffer combinations
> in the Emacs code base a bad idea, the solution is using grep and fixing
> all those occurences where you think they are an error.
Admittedly, there are too many occurrences of this idiom to be fixed by
a single person in reasonable time. I suppose Stefan, when fixing a bug
incorrectly using that idiom, decided that the respective authors should
care about the remaining instances. As a matter of fact, the majority
of them refuses to do so and the rest of us continue to live with the
annoying consequences.
IMHO, the best way to address this problem would have been to provide a
separate routine (which should be also able to catch occurrences like
(save-excursion
(if buffer
(set-buffer buffer))
in xml.el), get back to the various authors and ask them to fix these
issues.
martin
- Re: `save-excursion' defeated by `set-buffer', (continued)
- Re: `save-excursion' defeated by `set-buffer', Stefan Monnier, 2010/01/04
- Re: `save-excursion' defeated by `set-buffer', David Kastrup, 2010/01/05
- RE: `save-excursion' defeated by `set-buffer', Drew Adams, 2010/01/05
- Re: `save-excursion' defeated by `set-buffer', Stefan Monnier, 2010/01/05
- Re: `save-excursion' defeated by `set-buffer', David Kastrup, 2010/01/06
- RE: `save-excursion' defeated by `set-buffer', Drew Adams, 2010/01/06
- Re: `save-excursion' defeated by `set-buffer', Stefan Monnier, 2010/01/09
- Re: `save-excursion' defeated by `set-buffer', David Kastrup, 2010/01/10
- Re: `save-excursion' defeated by `set-buffer', Stefan Monnier, 2010/01/10
- Re: `save-excursion' defeated by `set-buffer', David Kastrup, 2010/01/11
- Re: `save-excursion' defeated by `set-buffer',
martin rudalics <=
- Re: `save-excursion' defeated by `set-buffer', Stefan Monnier, 2010/01/11
- RE: `save-excursion' defeated by `set-buffer', Drew Adams, 2010/01/10
- Re: `save-excursion' defeated by `set-buffer', Stefan Monnier, 2010/01/09
- Re: `save-excursion' defeated by `set-buffer', Harald Hanche-Olsen, 2010/01/10
- Re: `save-excursion' defeated by `set-buffer', martin rudalics, 2010/01/10
- RE: `save-excursion' defeated by `set-buffer', Drew Adams, 2010/01/10
- Re: `save-excursion' defeated by `set-buffer', Harald Hanche-Olsen, 2010/01/10
Re: `save-excursion' defeated by `set-buffer', Davis Herring, 2010/01/04