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

[Dan Nicolaescu]

Stefan Monnier <address@hidden> writes:

  > > Byte compiling this:
  > 
  > > (if (and (featurep 'xemacs) blah)
  > >     (foo))
  > 
  > > with emacs-22.1 does not result in any warning, doing it with the
  > > emacs from trunk results in this:
  > 
  > > In end of data:
  > > foo.el:2:11:Warning: the function `foo' is not known to be defined.
  > 
  > > Any idea what went wrong?
  > 
  > Stupid typo.  Sorry.  Should be fixed now,

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 ...)