Re: (if (and (featurep 'xemacs) blah) not optimized anymore

[Dan Nicolaescu]

Stefan Monnier <address@hidden> writes:

  > > I still see one issue:
  > 
  > > After applying this patch: 
  > 
  > > --- viper-cmd.el  19 Aug 2007 13:47:07 -0000      1.63
  > > +++ viper-cmd.el  22 Oct 2007 05:37:24 -0000
  > > @@ -861,7 +861,7 @@
  > >                               (1- (length quail-current-str)))))
  > >            ))
  > >         ((and viper-special-input-method
  > > -             viper-xemacs-p
  > > +             nil
  > >               (fboundp 'quail-start-translation))
  > >          ;; same as above but for XEmacs, which doesn't have
  > >          ;; quail-input-method
  > 
  > > the byte compiler will still warn about about quail-start-translation
  > > not being defined. I won't warn if the expression is 
  > > (and nil viper-special-input-method ...)
  > 
  > The patch below seems to fix it, but it does a bit more and I'm not sure
  > if it behaves well in practice (e.g. performancewise), so please try it
  > out on a large scale.  If you find it to work well, feel free to
  > install it.

Thanks!
The bootstrap time from a clean state went from 400s to 402s, so I
checked this in.