Re: message in *Compile Log* buffer

* David Kastrup (2005-01-20) writes:

> Ralf Angeli <[email protected]> writes:
>
>> The problem is `minor-mode-list'.  And this variable is only present
>> in CVS Emacs, not in Emacs 21.3 and earlier and not in XEmacs.  I
>> thought that checking for `(boundp 'minor-mode-list)' would suffice
>> to silence the byte compiler, but in some cases obviously it isn't.
>
> Well, in recent versions of Emacs CVS, it should be sufficient.  But
> it is not the recent versions that are missing minor-mode-list.
>
> What you can do is just
>
> (defvar minor-mode-list)
>
> at the top of the file.  That has no side effects except for silencing
> the byte compiler.

Okay, thanks for the hint.  But I think I'll just leave the code
as-is.  I don't think the number and definition of minor modes will
change frequently and thus maintenance of a list of minor modes in the
advice will rarely be needed.  In addition I am not really sure about
the security implications of calling minor mode functions.  With an
explicit list we are a bit more conservative compared to a match with
a regular expression.

-- 
Ralf