[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: What happened to (defun x)?
From: |
Dan Nicolaescu |
Subject: |
Re: What happened to (defun x)? |
Date: |
Sat, 24 Nov 2007 15:06:26 -0800 |
David Kastrup <address@hidden> writes:
> Dan Nicolaescu <address@hidden> writes:
>
> > address@hidden (Mark A. Hershberger) writes:
> >
> > > I have just merged a minimally tested nXML. I merged all the
> > > char-name/unicode/*.el files for now because, while this information
is
> > > in the unicode branch, it isn't in HEAD. That still needs to be
sorted
> > > out.
> >
> > Could you please fix the byte compile warnings? We are trying to
> > eliminate all the byte compiler warnings.
> > The free variable and defsubst warnings have been fixed in the rest of
> > emacs code.
> > Also it looks like all the the char-name/unicode/*.el files need a
> > (declare-function nxml-define-char-name-set at the start.
>
> I might have missed the discussion: pre-22 we had considered having
> (defun nxml-define-char-name-set)
> be a byte-compiler silencer in the same manner as
> (defvar preview-version)
> is a byte-compiler silencer. The obvious advantage over
> "declare-function" is that one does not need to remember another idiom
> and name.
>
> Is there a particular advantage for a separate declare-function that I
> just am not able to see?
`declare-function' was the result of a long, long discussion on this
list. Please read that discussion and the reasons this design was chosen
before restarting the same discussion again. At least restart the
discussion from an informed position.
- Re: initial nxml merge, (continued)