[CEDET-devel] Quick fix for JDEE and ECB in Emacs CVS

[Bryan W. <bry...@gm...>]

It looks like there's an issue with JDEE and ECB using bytecomp when using recent emacs cvs builds (I've tested with the 2008-12-05 build for ubuntu 8.10 from https://launchpad.net/~ubuntu-elisp/+archive and a custom build from today). Trying to load either results in a failure with "wrong number of arguments" error, and debug-on-entry points to a number of "byte-compile" and "byte-optimize" function calls just before the error. Since I figured that other people may be having the same problem, I've got a quick patch that may help. It's dumped from a git repo though, so it may need a bit of filename tweaking to apply cleanly to the cvs version though.

----------------------------

diff --git a/eieio/eieio.el b/eieio/eieio.el
index 1c5cb9d..8a34410 100755
--- a/eieio/eieio.el
+++ b/eieio/eieio.el
@@ -1197,11 +1197,11 @@ IMPL is the symbol holding the method implementation."
   ;; NOTE: I tried out byte compiling this little fcn.  Turns out it
   ;; is faster to execute this for not byte-compiled.  ie, install this,
   ;; then measure calls going through here.  I wonder why.
-  (require 'bytecomp)
-  (let ((byte-compile-free-references nil)
-	(byte-compile-warnings nil)
-	)
-    (byte-compile-lambda
+  ;; (require 'bytecomp)
+  ;; (let ((byte-compile-free-references nil)
+  ;;       (byte-compile-warnings nil)
+  ;;       )
+  ;;   (byte-compile-lambda
      `(lambda (&rest local-args)
 	,doc-string
 	;; This is a cool cheat.  Usually we need to look up in the
@@ -1234,7 +1234,7 @@ IMPL is the symbol holding the method implementation."
 	      (apply ,(list 'quote impl) local-args)
 	      ;(,impl local-args)
 	      ))))
-     ))
+     ;; ))  
   )
 
 (defsubst eieio-defgeneric-reset-generic-form-primary-only-one (method)