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

[Dan Nicolaescu <dann@ics.uci.edu>]

Stefan Monnier <monnier@iro.umontreal.ca> 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.