Re: save-excursion and the mark

[Stefan Monnier <monnier@IRO.UMontreal.CA>]

> One thing that I really didn't like is that I had to replace
>     (region-active-p)
> with 
>     (and mark-active (not deactivate-mark))

I think this should be

     (and (region-active-p) (not deactivate-mark))

The (not deactivate-mark) test has always been needed if you intend your
tests to check what the user would see after running that code
interactively (in which case the command loop would deactivate the mark
at the end of the command if deactivate-mark is set).

The fact that you did not need (not deactivate-mark) in the past was
just a lucky accident for the code you happened to test.


        Stefan