Re: [jdee-users] new JDEE release/patches (with CEDET 1.0pre6) ?
Brought to you by:
paullandes
From: Paul L. <la...@ma...> - 2009-06-04 04:38:25
|
I'll look into it. Vincent Belaïche writes: > > Yes, this is a patch number from SourceForge. I am not the one who submitted the patch, the patch is not very precise on what to do. Myself, I don't know whether the work-around which I made is correct or not. Maybe it is just hiding a more general problem. > > What happens is that I am debugging jPicEdt (a LaTeX vector drawing editor to which development team of I belong) with JDEE environnement. When I launch the debugger with `C-c C-v C-d' the function jde-parse-class-exists is called with argument "jpicedt.JPicEdt" (that is to say the main class), then this calls > jde-jeval-r with argument "jde.util.JdeUtilities.classExists(\"jpicedt.JpicEdt\");", which in turn returns `nil'. > > Maybe the problem is not in function jde-find-class-source-file, but rather in the fact that the jde-jeval-r does not return a correct value. I don't know. Anyhow, the function jde-find-class-source-file looks strange because function name jde-parse-class-exists suggests a boolean return. > > Please feel free to get in touch with me again if you need more information on how the problem happens. > > I think that it would be more appropriate to submit a bug, than a patch, because I am not sure that this patch is a correct correction. > > Best Regards, > > Vincent. > > > > From: la...@ma... > > Date: Sun, 31 May 2009 21:11:08 -0500 > > To: vin...@ho... > > CC: jde...@li... > > Subject: Re: [jdee-users] new JDEE release/patches (with CEDET 1.0pre6) ? > > > > Is this a SF bug tracker number? Please send the patch again. > > > > Thanks. > > > > Vincent Belaïche writes: > > > > > > > > > Now I have made this update, and re byte-compiled all files, there seems to be a problem with debugging : it seems that I have to apply patch 1861505 again, that it is not taken in the latest version... > > > > > > I have made the patch as follows : > > > > > > (defun jde-find-class-source-file (class) > > > "... DOCSTRING ..." ;<-- ABBREVIATED > > > (let* ((verified-name > > > (or ; <-- PATCH LINE > > > (jde-parse-class-exists class) > > > class); <-- PATCH LINE > > > ) > > > (outer-class (jde-remove-inner-class verified-name)) > > > ;<-- ABBREVIATED > > > > > > Vincent. > > > > > > > > > > > > From: vin...@ho... > > > To: ht...@in...; er...@si... > > > Date: Sun, 31 May 2009 22:39:35 +0200 > > > CC: jde...@li... > > > Subject: Re: [jdee-users] new JDEE release/patches (with CEDET 1.0pre6) ? > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > I installed SVN on my machine, and got the latest elisp code. At least now it seems to compile. > > > > > > Thanks... > > > Vincent. > > > > > > > > > > To: er...@si... > > > > CC: vin...@ho...; jde...@li... > > > > Subject: Re: [jdee-users] new JDEE release/patches (with CEDET 1.0pre6) ? > > > > From: ht...@in... > > > > Date: Sun, 31 May 2009 17:02:59 +0100 > > > > > > > > -----BEGIN PGP SIGNED MESSAGE----- > > > > Hash: SHA1 > > > > > > > > Eric M. Ludlam writes: > > > > > > > > > > > > > that never ran. When a function turned into a macro in EIEIO, it causes > > > > > the below error during byte compilation. That error is fixed in JDE > > > > > somewhere. > > > > > > > > Indeed it's been fixed in CVS for some time. Here is the patch, for > > > > those who can't/won't use the repository: > > > > > > > > ht > > > > > > > > 650,651c650,654 > > > > < (error "Tried to add an item of type %s to a list of items of > > > > type %s" > > > > < (typep item) (oref this elem-type)))) > > > > - --- > > > > > (error "efc-list error: tried to add an item of type %s to a > > > > list of items of type %s" > > > > > (if (object-p item) > > > > > (object-class item) > > > > > (type-of item)) > > > > > (oref this elem-type)))) > > > > 763c766 > > > > < (oset table (make-hash-table))) > > > > - --- > > > > > (oset this table (make-hash-table))) > > > > 770c773,776 > > > > < (typep value) (oref this elem-type)))) > > > > - --- > > > > > (if (object-p value) > > > > > (object-class value) > > > > > (type-of value)) > > > > > (oref this elem-type)))) > > > > > > > > - -- > > > > Henry S. Thompson, School of Informatics, University of Edinburgh > > > > Half-time member of W3C Team > > > > 10 Crichton Street, Edinburgh EH8 9AB, SCOTLAND -- (44) 131 650-4440 > > > > Fax: (44) 131 651-1426, e-mail: ht...@in... > > > > URL: http://www.ltg.ed.ac.uk/~ht/ > > > > [mail really from me _always_ has this .sig -- mail without it is forged spam] > > > > -----BEGIN PGP SIGNATURE----- > > > > Version: GnuPG v1.2.6 (GNU/Linux) > > > > > > > > iD8DBQFKIqozkjnJixAXWBoRAsYdAJ9BmfGoHCsguHxEW8R+Vqz6Ye2Q3gCfatIZ > > > > 9eNvqQUdEAK5Npmk3JKctlM= > > > > =TQDk > > > > -----END PGP SIGNATURE----- > > > > > > Souhaitez vous « être au bureau sans y être » ? Oui je le veux ! > > > _________________________________________________________________ > > > Découvrez toutes les possibilités de communication avec vos proches > > > http://www.microsoft.com/windows/windowslive/default.aspx<html> > > > <head> > > > <style> > > > .hmmessage P > > > { > > > margin:0px; > > > padding:0px > > > } > > > body.hmmessage > > > { > > > font-size: 10pt; > > > font-family:Verdana > > > } > > > </style> > > > </head> > > > <body class='hmmessage'> > > > <br><font style="font-size: 10pt;" size="2"><font style="" face="Arial">Now I have made this update, and re byte-compiled all files, there seems to be a problem with debugging : it seems that I have to apply patch 1861505 again, that it is not taken in the latest version...<br><br>I have made the patch as follows :<br><br>(defun jde-find-class-source-file (class)<br> "... DOCSTRING ..." ;<-- ABBREVIATED<br> (let* ((verified-name <br> (or ; <-- PATCH LINE<br> (jde-parse-class-exists class)<br> class); <-- PATCH LINE<br> )<br> (outer-class (jde-remove-inner-class verified-name))<br></font></font><font style="font-size: 10pt;" size="2"><font style="" face="Arial"> ;<-- ABBREVIATED</font></font><br><font style="font-size: 10pt;" size="2"><font style="" face="Arial"><br> Vincent.<br></font></font><span><strong><br><br><br></strong></span><hr id="stopSpelling">From: vin...@ho...<br>To: ht...@in...; er...@si...<br>Date: Sun, 31 May 2009 22:39:35 +0200<br>CC: jde...@li...<br>Subject: Re: [jdee-users] new JDEE release/patches (with CEDET 1.0pre6) ?<br><br> > > > > > > > > > > > > <style> > > > .ExternalClass .EC_hmmessage P > > > {padding:0px;} > > > .ExternalClass body.EC_hmmessage > > > {font-size:10pt;font-family:Verdana;} > > > </style> > > > > > > > > > <br>I installed SVN on my machine, and got the latest elisp code. At least now it seems to compile.<br><br>Thanks...<br> Vincent.<br> <br><br>> To: er...@si...<br>> CC: vin...@ho...; jde...@li...<br>> Subject: Re: [jdee-users] new JDEE release/patches (with CEDET 1.0pre6) ?<br>> From: ht...@in...<br>> Date: Sun, 31 May 2009 17:02:59 +0100<br>> <br>> -----BEGIN PGP SIGNED MESSAGE-----<br>> Hash: SHA1<br>> <br>> Eric M. Ludlam writes:<br>> <br>> <br>> > that never ran. When a function turned into a macro in EIEIO, it causes<br>> > the below error during byte compilation. That error is fixed in JDE<br>> > somewhere.<br>> <br>> Indeed it's been fixed in CVS for some time. Here is the patch, for<br>> those who can't/won't use the repository:<br>> <br>> ht<br>> <br>> 650,651c650,654<br>> < (error "Tried to add an item of type %s to a list of items of<br>> type %s"<br>> < (typep item) (oref this elem-type))))<br>> - ---<br>> > (error "efc-list error: tried to add an item of type %s to a<br>> list of items of type %s"<br>> > (if (object-p item)<br>> > (object-class item)<br>> > (type-of item))<br>> > (oref this elem-type))))<br>> 763c766<br>> < (oset table (make-hash-table)))<br>> - ---<br>> > (oset this table (make-hash-table)))<br>> 770c773,776<br>> < (typep value) (oref this elem-type))))<br>> - ---<br>> > (if (object-p value)<br>> > (object-class value)<br>> > (type-of value))<br>> > (oref this elem-type))))<br>> <br>> - -- <br>> Henry S. Thompson, School of Informatics, University of Edinburgh<br>> Half-time member of W3C Team<br>> 10 Crichton Street, Edinburgh EH8 9AB, SCOTLAND -- (44) 131 650-4440<br>> Fax: (44) 131 651-1426, e-mail: ht...@in...<br>> URL: http://www.ltg.ed.ac.uk/~ht/<br>> [mail really from me _always_ has this .sig -- mail without it is forged spam]<br>> -----BEGIN PGP SIGNATURE-----<br>> Version: GnuPG v1.2.6 (GNU/Linux)<br>> <br>> iD8DBQFKIqozkjnJixAXWBoRAsYdAJ9BmfGoHCsguHxEW8R+Vqz6Ye2Q3gCfatIZ<br>> 9eNvqQUdEAK5Npmk3JKctlM=<br>> =TQDk<br>> -----END PGP SIGNATURE-----<br><br><hr>Souhaitez vous « être au bureau sans y être » ? <a href="http://www.microsoft.com/france/windows/bts/default.mspx">Oui je le veux !</a><br /><hr />Découvrez toutes les possibilités de communication <a href='http://www.microsoft.com/windows/windowslive/default.aspx' target='_new'>avec vos proches</a></body> > > > </html>------------------------------------------------------------------------------ > > > Register Now for Creativity and Technology (CaT), June 3rd, NYC. CaT > > > is a gathering of tech-side developers & brand creativity professionals. Meet > > > the minds behind Google Creative Lab, Visual Complexity, Processing, & > > > iPhoneDevCamp as they present alongside digital heavyweights like Barbarian > > > Group, R/GA, & Big Spaceship. http://p.sf.net/sfu/creativitycat-com _______________________________________________ > > > jdee-users mailing list > > > jde...@li... > > > https://lists.sourceforge.net/lists/listinfo/jdee-users > > > > > > -- > > Paul Landes > > la...@ma... > > _________________________________________________________________ > Découvrez toutes les possibilités de communication avec vos proches > http://www.microsoft.com/windows/windowslive/default.aspx<html> > <head> > <style> > .hmmessage P > { > margin:0px; > padding:0px > } > body.hmmessage > { > font-size: 10pt; > font-family:Verdana > } > </style> > </head> > <body class='hmmessage'> > Yes, this is a patch number from SourceForge. I am not the one who submitted the patch, the patch is not very precise on what to do. Myself, I don't know whether the work-around which I made is correct or not. Maybe it is just hiding a more general problem.<br><br>What happens is that I am debugging jPicEdt (a LaTeX vector drawing editor to which development team of I belong) with JDEE environnement. When I launch the debugger with `C-c C-v C-d' the function jde-parse-class-exists is called with argument "jpicedt.JPicEdt" (that is to say the main class), then this calls <br>jde-jeval-r with argument "jde.util.JdeUtilities.classExists(\"jpicedt.JpicEdt\");", which in turn returns `nil'.<br><br>Maybe the problem is not in function jde-find-class-source-file, but rather in the fact that the jde-jeval-r does not return a correct value. I don't know. Anyhow, the function jde-find-class-source-file looks strange because function name jde-parse-class-exists suggests a boolean return.<br><br>Please feel free to get in touch with me again if you need more information on how the problem happens.<br><br>I think that it would be more appropriate to submit a bug, than a patch, because I am not sure that this patch is a correct correction.<br><br>Best Regards,<br><br> Vincent.<br><br><br>> From: la...@ma...<br>> Date: Sun, 31 May 2009 21:11:08 -0500<br>> To: vin...@ho...<br>> CC: jde...@li...<br>> Subject: Re: [jdee-users] new JDEE release/patches (with CEDET 1.0pre6) ?<br>> <br>> Is this a SF bug tracker number? Please send the patch again.<br>> <br>> Thanks.<br>> <br>> Vincent Belaïche writes:<br>> > <br>> > <br>> > Now I have made this update, and re byte-compiled all files, there seems to be a problem with debugging : it seems that I have to apply patch 1861505 again, that it is not taken in the latest version...<br>> > <br>> > I have made the patch as follows :<br>> > <br>> > (defun jde-find-class-source-file (class)<br>> > "... DOCSTRING ..." ;<-- ABBREVIATED<br>> > (let* ((verified-name <br>> > (or ; <-- PATCH LINE<br>> > (jde-parse-class-exists class)<br>> > class); <-- PATCH LINE<br>> > )<br>> > (outer-class (jde-remove-inner-class verified-name))<br>> > ;<-- ABBREVIATED<br>> > <br>> > Vincent.<br>> > <br>> > <br>> > <br>> > From: vin...@ho...<br>> > To: ht...@in...; er...@si...<br>> > Date: Sun, 31 May 2009 22:39:35 +0200<br>> > CC: jde...@li...<br>> > Subject: Re: [jdee-users] new JDEE release/patches (with CEDET 1.0pre6) ?<br>> > <br>> > <br>> > <br>> > <br>> > <br>> > <br>> > <br>> > <br>> > <br>> > I installed SVN on my machine, and got the latest elisp code. At least now it seems to compile.<br>> > <br>> > Thanks...<br>> > Vincent.<br>> > <br>> > <br>> > > To: er...@si...<br>> > > CC: vin...@ho...; jde...@li...<br>> > > Subject: Re: [jdee-users] new JDEE release/patches (with CEDET 1.0pre6) ?<br>> > > From: ht...@in...<br>> > > Date: Sun, 31 May 2009 17:02:59 +0100<br>> > > <br>> > > -----BEGIN PGP SIGNED MESSAGE-----<br>> > > Hash: SHA1<br>> > > <br>> > > Eric M. Ludlam writes:<br>> > > <br>> > > <br>> > > > that never ran. When a function turned into a macro in EIEIO, it causes<br>> > > > the below error during byte compilation. That error is fixed in JDE<br>> > > > somewhere.<br>> > > <br>> > > Indeed it's been fixed in CVS for some time. Here is the patch, for<br>> > > those who can't/won't use the repository:<br>> > > <br>> > > ht<br>> > > <br>> > > 650,651c650,654<br>> > > < (error "Tried to add an item of type %s to a list of items of<br>> > > type %s"<br>> > > < (typep item) (oref this elem-type))))<br>> > > - ---<br>> > > > (error "efc-list error: tried to add an item of type %s to a<br>> > > list of items of type %s"<br>> > > > (if (object-p item)<br>> > > > (object-class item)<br>> > > > (type-of item))<br>> > > > (oref this elem-type))))<br>> > > 763c766<br>> > > < (oset table (make-hash-table)))<br>> > > - ---<br>> > > > (oset this table (make-hash-table)))<br>> > > 770c773,776<br>> > > < (typep value) (oref this elem-type))))<br>> > > - ---<br>> > > > (if (object-p value)<br>> > > > (object-class value)<br>> > > > (type-of value))<br>> > > > (oref this elem-type))))<br>> > > <br>> > > - -- <br>> > > Henry S. Thompson, School of Informatics, University of Edinburgh<br>> > > Half-time member of W3C Team<br>> > > 10 Crichton Street, Edinburgh EH8 9AB, SCOTLAND -- (44) 131 650-4440<br>> > > Fax: (44) 131 651-1426, e-mail: ht...@in...<br>> > > URL: http://www.ltg.ed.ac.uk/~ht/<br>> > > [mail really from me _always_ has this .sig -- mail without it is forged spam]<br>> > > -----BEGIN PGP SIGNATURE-----<br>> > > Version: GnuPG v1.2.6 (GNU/Linux)<br>> > > <br>> > > iD8DBQFKIqozkjnJixAXWBoRAsYdAJ9BmfGoHCsguHxEW8R+Vqz6Ye2Q3gCfatIZ<br>> > > 9eNvqQUdEAK5Npmk3JKctlM=<br>> > > =TQDk<br>> > > -----END PGP SIGNATURE-----<br>> > <br>> > Souhaitez vous « être au bureau sans y être » ? Oui je le veux !<br>> > _________________________________________________________________<br>> > Découvrez toutes les possibilités de communication avec vos proches<br>> > http://www.microsoft.com/windows/windowslive/default.aspx<html><br>> > <head><br>> > <style><br>> > .hmmessage P<br>> > {<br>> > margin:0px;<br>> > padding:0px<br>> > }<br>> > body.hmmessage<br>> > {<br>> > font-size: 10pt;<br>> > font-family:Verdana<br>> > }<br>> > </style><br>> > </head><br>> > <body class='hmmessage'><br>> > <br><font style="font-size: 10pt;" size="2"><font style="" face="Arial">Now I have made this update, and re byte-compiled&nbsp; all files, there seems to be a problem with debugging : it seems that I have to apply patch&nbsp; 1861505 again, that it is not taken in the latest version...<br><br>I have made the patch as follows :<br><br>(defun jde-find-class-source-file (class)<br>&nbsp; "... DOCSTRING ..." ;&lt;-- ABBREVIATED<br>&nbsp; (let* ((verified-name <br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; (or ; &lt;-- PATCH LINE<br>&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;(jde-parse-class-exists class)<br>&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;class); &lt;-- PATCH LINE<br>&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;)<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; (outer-class (jde-remove-inner-class verified-name))<br></font></font><font style="font-size: 10pt;" size="2"><font style="" face="Arial"> ;&lt;-- ABBREVIATED</font></font><br><font style="font-size: 10pt;" size="2"><font style="" face="Arial"><br>&nbsp;Vincent.<br></font></font><span><strong><br><br><br></strong></span><hr id="stopSpelling">From: vin...@ho...<br>To: ht...@in...; er...@si...<br>Date: Sun, 31 May 2009 22:39:35 +0200<br>CC: jde...@li...<br>Subject: Re: [jdee-users] new JDEE release/patches (with CEDET 1.0pre6) ?<br><br><br>> > <br>> > <br>> > <br>> > <style><br>> > .ExternalClass .EC_hmmessage P<br>> > {padding:0px;}<br>> > .ExternalClass body.EC_hmmessage<br>> > {font-size:10pt;font-family:Verdana;}<br>> > </style><br>> > <br>> > <br>> > <br>I installed SVN on my machine,&nbsp; and got the latest elisp code.&nbsp; At least now it seems to compile.<br><br>Thanks...<br>&nbsp;&nbsp;&nbsp; Vincent.<br>&nbsp;&nbsp; <br><br>&gt; To: er...@si...<br>&gt; CC: vin...@ho...; jde...@li...<br>&gt; Subject: Re: [jdee-users] new JDEE release/patches (with CEDET 1.0pre6) ?<br>&gt; From: ht...@in...<br>&gt; Date: Sun, 31 May 2009 17:02:59 +0100<br>&gt; <br>&gt; -----BEGIN PGP SIGNED MESSAGE-----<br>&gt; Hash: SHA1<br>&gt; <br>&gt; Eric M. Ludlam writes:<br>&gt; <br>&gt; <br>&gt; &gt; that never ran. When a function turned into a macro in EIEIO, it causes<br>&gt; &gt; the below error during byte compilation. That error is fixed in JDE<br>&gt; &gt; somewhere.<br>&gt; <br>&gt; Indeed it's been fixed in CVS for some time. Here is the patch, for<br>&gt; those who can't/won't use the repository:<br>&gt; <br>&gt; ht<br>&gt; <br>&gt; 650,651c650,654<br>&gt; &lt; (error "Tried to add an item of type %s to a list of items of<br>&gt; type %s"<br>&gt; &lt; (typep item) (oref this elem-type))))<br>&gt; - ---<br>&gt; &gt; (error "efc-list error: tried to add an item of type %s to a<br>&gt; list of items of type %s"<br>&gt; &gt; (if (object-p item)<br>&gt; &gt; (object-class item)<br>&gt; &gt; (type-of item))<br>&gt; &gt; (oref this elem-type))))<br>&gt; 763c766<br>&gt; &lt; (oset table (make-hash-table)))<br>&gt; - ---<br>&gt; &gt; (oset this table (make-hash-table)))<br>&gt; 770c773,776<br>&gt; &lt; (typep value) (oref this elem-type))))<br>&gt; - ---<br>&gt; &gt; (if (object-p value)<br>&gt; &gt; (object-class value)<br>&gt; &gt; (type-of value))<br>&gt; &gt; (oref this elem-type))))<br>&gt; <br>&gt; - -- <br>&gt; Henry S. Thompson, School of Informatics, University of Edinburgh<br>&gt; Half-time member of W3C Team<br>&gt; 10 Crichton Street, Edinburgh EH8 9AB, SCOTLAND -- (44) 131 650-4440<br>&gt; Fax: (44) 131 651-1426, e-mail: ht...@in...<br>&gt; URL: http://www.ltg.ed.ac.uk/~ht/<br>&gt; [mail really from me _always_ has this .sig -- mail without it is forged spam]<br>&gt; -----BEGIN PGP SIGNATURE-----<br>&gt; Version: GnuPG v1.2.6 (GNU/Linux)<br>&gt; <br>&gt; iD8DBQFKIqozkjnJixAXWBoRAsYdAJ9BmfGoHCsguHxEW8R+Vqz6Ye2Q3gCfatIZ<br>&gt; 9eNvqQUdEAK5Npmk3JKctlM=<br>&gt; =TQDk<br>&gt; -----END PGP SIGNATURE-----<br><br><hr>Souhaitez vous &nbsp;«&nbsp;être au bureau sans y être&nbsp;»&nbsp;? <a href="http://www.microsoft.com/france/windows/bts/default.mspx">Oui je le veux !</a><br /><hr />Découvrez toutes les possibilités de communication <a href='http://www.microsoft.com/windows/windowslive/default.aspx' target='_new'>avec vos proches</a></body><br>> > </html>------------------------------------------------------------------------------<br>> > Register Now for Creativity and Technology (CaT), June 3rd, NYC. CaT <br>> > is a gathering of tech-side developers & brand creativity professionals. Meet<br>> > the minds behind Google Creative Lab, Visual Complexity, Processing, & <br>> > iPhoneDevCamp as they present alongside digital heavyweights like Barbarian <br>> > Group, R/GA, & Big Spaceship. http://p.sf.net/sfu/creativitycat-com _______________________________________________<br>> > jdee-users mailing list<br>> > jde...@li...<br>> > https://lists.sourceforge.net/lists/listinfo/jdee-users<br>> <br>> <br>> -- <br>> Paul Landes<br>> la...@ma...<br><br /><hr />Découvrez toutes les possibilités de communication <a href='http://www.microsoft.com/windows/windowslive/default.aspx' target='_new'>avec vos proches</a></body> > </html> -- Paul Landes la...@ma... |