bug#5294: 23.1; unload-feature disable minor-mode

[Kevin Ryde]

Juanma Barranquero <lekktu@gmail.com> writes:
>
> In fact, is a hook run *instead* of some other code.

Ah, I didn't know that.

> The very act of
> defining FOO-unload-hook has unexpected consequences: `unlead-feature'
> won't remove FOO functions from hooks, nor from `auto-mode-alist'. And
> it was always so; that's why I pushed for introducing
> FOO-unload-function.

Sounds like it could have been better to do those latter actions
irrespective of the hook.  Let only the FOO-unload-function say not to
do them, for the rare cases they're unwanted.