emacs-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: declare-function in files from Gnus (hashcash.el, imap.el)


From: Dan Nicolaescu
Subject: Re: declare-function in files from Gnus (hashcash.el, imap.el)
Date: Sat, 01 Dec 2007 08:34:07 -0800

Ralf Angeli <address@hidden> writes:

  > * Reiner Steib (2007-11-30) writes:
  > 
  > > I think the following changes...
  > >
  > > 2007-11-27  Dan Nicolaescu  <address@hidden>
  > > [...]
  > >   * mail/hashcash.el (message-narrow-to-headers-or-head)
  > >   (message-fetch-field, message-goto-eoh)
  > >   (message-narrow-to-headers):
  > >   [...] Declare as functions.
  > >
  > > 2007-11-29  Glenn Morris  <address@hidden>
  > > [...]
  > >   * net/imap.el (sasl-find-mechanism, sasl-mechanism-name)
  > >   (sasl-make-client, sasl-next-step, sasl-step-data)
  > >   (sasl-step-set-data)
  > >   [...] Declare as functions.
  > >
  > > ... will break compatibility of Gnus with Emacs 21, Emacs 22.1 and
  > > XEmacs when synced to Gnus repository (doesn't happen immediately,
  > > because Miles is on a trip).  
  > >
  > > The files came from Gnus and are only used by Gnus, AFAIK.  We could
  > > probably skip these changes when syncing, keeping the files identical
  > > simplifies maintenance.
  > >
  > > Maybe we could add (defalias 'declare-function 'ignore ...) (like in
  > > Emacs 22.2) to `gnus/lisp/dgnushack.el' (in Gnus).
  > 
  > There is the same problem with RefTeX.
  > 
  > Can somebody explain why these `declare-function' calls are needed?

`declare-function' has been discussed here at length. It is used to
avoid the "unknown function" warnings produced by the byte compiler.

Talking about RefTeX, can you please fix this warning:

In reftex-make-separate-toc-frame:
reftex-toc.el:998:35:Warning: `focus-frame' is an obsolete function (as
of  Emacs 22.1); it does nothing.




reply via email to

[Prev in Thread] Current Thread [Next in Thread]