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: lan...@mailc.net
> Date: Sun, 31 May 2009 21:11:08 -0500
> To: vincent....@hotmail.fr
> CC: jdee-users@lists.sourceforge.net
> 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: vincent....@hotmail.fr
> > To: h...@inf.ed.ac.uk; e...@siege-engine.com
> > Date: Sun, 31 May 2009 22:39:35 +0200
> > CC: jdee-users@lists.sourceforge.net
> > 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: e...@siege-engine.com
> > > CC: vincent....@hotmail.fr; jdee-users@lists.sourceforge.net
> > > Subject: Re: [jdee-users] new JDEE release/patches (with CEDET 1.0pre6) ?
> > > From: h...@inf.ed.ac.uk
> > > 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: h...@inf.ed.ac.uk
> > > 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: vincent....@hotmail.fr<br>To: h...@inf.ed.ac.uk;
> e...@siege-engine.com<br>Date: Sun, 31 May 2009 22:39:35 +0200<br>CC:
> jdee-users@lists.sourceforge.net<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: e...@siege-engine.com<br>> CC: vincent....@hotmail.fr;
> jdee-users@lists.sourceforge.net<br>> Subject: Re: [jdee-users] new JDEE
> release/patches (with CEDET 1.0pre6) ?<br>> From:
> h...@inf.ed.ac.uk<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: h...@inf.ed.ac.uk<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
> > jdee-users@lists.sourceforge.net
> > https://lists.sourceforge.net/lists/listinfo/jdee-users
>
>
> --
> Paul Landes
> lan...@mailc.net
_________________________________________________________________
Découvrez toutes les possibilités de communication avec vos proches
http://www.microsoft.com/windows/windowslive/default.aspx------------------------------------------------------------------------------
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
jdee-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/jdee-users
