Re: |PATCH| describe-minor-mode and describe-minor-mode-from-indicator

[Richard Stallman]

    I generally believe that `eval' should be avoided.  This is especially
    true here since you call `symbol-name' so you already assume that (car x)
    is a symbol, so you could just call `symbol-value' instead of `eval'.
    But note also that nothing guarantees you that (car x) is bound.
    Finally, I think it's perfectly OK (if not preferable) to list all
    the minor modes rather than just the currently active ones, so
    I'd just use

      (delq nil (mapcar (lambda (x) (symbol-name (car x))) minor-mode-alist))

minor-mode-alist can get quite long, so I think it should show only
the enabled ones.  Using boundp and symbol-value is a good approach.

I've alredy asked Juanma to install the code that Masataki sent.
How about if you install the improvements that you can see to make?