2 Feb 2008 23:58
ECB patch
Eric M. Ludlam <eric <at> siege-engine.com>
2008-02-02 22:58:28 GMT
2008-02-02 22:58:28 GMT
Howdy,
Last fall I optimized the analyzer, and a slot ECB was using was
deleted. The following patch (plus comment you can delete at your
leisure) uses a function that ought to work with older versions of
CEDET as well as the CVS version.
ECB users wrappers for all this stuff so it probably needs some
additional assistance.
I found a compile warning about a new required argument too, but I
made that optional in CEDET so that code can stay the same.
I gave it a quick run, and it works against CVS/CEDET.
Enjoy
Eric
---------------------------------
*** ecb-analyse.el 12 May 2006 12:03:11 -0400 1.16
--- ecb-analyse.el 02 Feb 2008 17:56:01 -0500
***************
*** 270,276 ****
(defmethod ecb-analyse-more-nodes ((context semantic-analyze-context))
"Show a set of ecb-nodes specific to CONTEXT."
! (let ((localvars (oref context localvariables)))
(when localvars
(ecb-analyse-add-nodes "Local Variables" "Local Variables" localvars
ecb-analyse-nodetype-localvars)))
(Continue reading)
Ciao,
Klaus
Eric M. Ludlam wrote:
> Howdy,
>
> Last fall I optimized the analyzer, and a slot ECB was using was
> deleted. The following patch (plus comment you can delete at your
> leisure) uses a function that ought to work with older versions of
> CEDET as well as the CVS version.
>
> ECB users wrappers for all this stuff so it probably needs some
> additional assistance.
>
> I found a compile warning about a new required argument too, but I
> made that optional in CEDET so that code can stay the same.
>
> I gave it a quick run, and it works against CVS/CEDET.
>
> Enjoy
> Eric
>
> ---------------------------------
> *** ecb-analyse.el 12 May 2006 12:03:11 -0400 1.16
> --- ecb-analyse.el 02 Feb 2008 17:56:01 -0500
> ***************
RSS Feed