Re: spam-stat warning

[Ted Zlatanov <tzz@lifelogs.com>]

On Tue, 31 May 2011 23:09:35 +0200 Lars Magne Ingebrigtsen <larsi@gnus.org> wrote: 

LMI> On each Gnus startup I now get the following warning in Emacs 24:
LMI> Warning: defvar ignored because spam-stat-install-hooks is let-bound

LMI> And this is the reason:

LMI> (eval-and-compile
LMI>   (when (condition-case nil
LMI>             (let ((spam-stat-install-hooks nil))
LMI>               (require 'spam-stat))
LMI>           (file-error
LMI>            (defalias 'spam-stat-register-ham-routine 'ignore)
LMI>            (defalias 'spam-stat-register-spam-routine 'ignore)
LMI>            nil))

LMI> And this is because:           
           
LMI> (when spam-stat-install-hooks
LMI>   (spam-stat-install-hooks-function))
           
LMI> I think that this may be a misfeature.  Just `require'-ing a file should
LMI> do anything but load the file, in my opinion.  If you want to install
LMI> stuff, then you should call the installation function explicitly.
LMI> Otherwise it's just too confusing...

I don't think many people are using spam-stat.el anyway, so changing
this immediately is OK with me.  Do you want:

(defun spam-stat-install ()
  (interactive)
  (when spam-stat-install-hooks
    (spam-stat-install-hooks-function)))

or were you thinking deeper surgery?

Ted