Re: Change in bytecomp.el breaks Gnus

[Nick Roberts <nickrob@snap.net.nz>]

 > Here is the compiler warning:
 > 
 >   Compiling file /home/teirllm/emacscvsdir/emacs/lisp/winner.el at Mon
 >   Nov 15 17:27:42 2004
 >   Entering directory `/home/teirllm/emacscvsdir/emacs/lisp/'
 >   winner.el:53:10:Warning: Function `gensym' from cl package called at runtime
 > 
 > This warning is bogus.  `gensym' can indeed be called at runtime, but
 > only if `setf' is called first, which requires cl to be loaded anyway.

I don't think this is bogus. setf is a macro and is not needed at runtime
because it has been expanded and compiled into winner.elc (I think you might
mean defsetf but that is also a macro, so the same applies). gensym, on the
other hand, is a function, so Emacs will need to know where to find it
when it is called.

The warning is presumably there to help the person writing the code decide
if cl needs to be loaded. Without it, it may be the user who discovers the
problem at runtime. Once the the coder decides its not a problem, then
putting with-no-warnings around the call would, indeed, appear to be the
right thing to do.

Nick