Hi,
I updated my Emacs from CVS recently (Dec 24) and have since done a
full rebuild of CEDET with the fresh Emacs.
I have run into no such problems during compilation of CEDET, nor
ECB from CVS. I didn't try JDEE . There have been a couple EIEIO
changes since your build of 12/05, so perhaps it was fixed?
I cannot install ubuntu but if you have some other straightforward
compilation steps I can take to repro the problem, I'd like to know
what they might be.
I don't think the byte-comp step should be removed from this code,
since it is a part of the performance boost the change went in for in
the first place.
Thanks
Eric
>>> Bryan Waite <bryanwaite@...> seems to think that:
>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)
>
>------------------------------------------------------------------------------
>_______________________________________________
>Cedet-devel mailing list
>Cedet-devel@...
>https://lists.sourceforge.net/lists/listinfo/cedet-devel
>
--
Eric Ludlam: eric@...
Siege: http://www.siege-engine.com Emacs: http://cedet.sourceforge.net
|