bug#25505: 24.5; doc of `define-minor-mode': incorrect for :keymap

[Drew Adams]

> tags 25505 notabug
> quit
> 
> The doc says "variable name" as opposed to "expression", I think it's
> clear that expression will be evaluated, and something which is not an
> expression will not be evaluated.  I guess we could add "it should be an
> unquoted variable name..."

I don't think the text is clear at all.  As you acknowledge, a macro
can, but need not, evaluate any of its arguments.  This macro evaluates
some and not others (like `setq', `defvar', etc.).  Its doc should be
explicit (aka clear) about which args are evaluated.  It is not clear now.