* Aidan Kehoe <kehoea(a)parhasard.net> writes:
Ar an ceathrú lá de mí Méan Fómhair, scríobh Steve Youngs:
> * Aidan Kehoe <kehoea(a)parhasard.net> writes:
> > Ar an dara lá de mí Méan Fómhair, scríobh Steve Youngs:
> >> APPROVE 21.5 RECOMMEND 21.4
> >> This patch optimises #'featurep for the cases...
> >> (featurep 'xemacs) and (featurep 'sxemacs)
> >> ...making the former be always t and the latter be nil.
> > The latter behaviour is inappropriate, isn’t it?
> Not at all. You don't want (featurep 'sxemacs) returning t on XEmacs do
Nope. But it shouldn’t be optimised away to nil if the code is to run
SXEmacs, though! Test case, with your patch applied to XEmacs:
$ echo "(if (featurep 'sxemacs) (message \"in
SXEmacs\") (message \"not\"))" > /tmp/whatever.el
$ ~/xemacs-21.5/src/xemacs -batch -f batch-byte-compile /tmp/whatever.el
$ ~/sxemacs-22.1.6/src/sxemacs -batch -l /tmp/whatever.elc
Oh hell! You are absolutely right. And it would be just as bad going
the other way... compile with SXE and run with XE.
Here's the patch to revert it, could somebody please apply it for me
2007-09-04 Steve Youngs <steve(a)sxemacs.org>
* byte-optimize.el (byte-optimize-featurep): Remove. It's wrong.
|---<Steve Youngs>---------------<GnuPG KeyID: A94B3003>---|
| SXEmacs - The only _______ you'll ever need. |
| Fill in the blank, yes, it's THAT good! |
XEmacs-Patches mailing list