Re: custom-set-minor-mode

[Richard Stallman <rms@gnu.org>]

    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.