Re: [PATCH] delete-trailing-whitespace on active region

[Stefan Monnier]

> +  (interactive "*r")

Using "r" and then checking region-active-p sounds clever, but is wrong:
- "r" will fail if the mark has never been set in the current buffer.
- checking region-active-p is wrong in non-interactive use.
I.e. the `interactive' spec should do the region-active-p check.
Oh, and it should use use-region-p instead of region-active-p.


        Stefan