Otherwise Custom might get confused if something "simplifies" the non-nil value to `t', as I believe `define-minor-mode' does. On the other hand, it is clearly documented that any non-nil value of the variable indicates an enabled mode.
Yes, any non-nil value *in the variable* should enable the mode. So your change is clearly correct. However, 0 as argument to the minor mode function is supposed to turn the mode off. Could you verify that still works right afetr your change? If it doesn't, another change in another place is probably needed. _______________________________________________ Emacs-devel mailing list Emacs-devel@gnu.org http://lists.gnu.org/mailman/listinfo/emacs-devel