Bug 120903 - VerifyError iff in AJDT using around advice
Summary: VerifyError iff in AJDT using around advice
Status: RESOLVED FIXED
Alias: None
Product: AJDT
Classification: Tools
Component: Core (show other bugs)
Version: DEVELOPMENT   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: AJDT-inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-12-14 11:49 EST by Wes Isberg CLA
Modified: 2006-04-13 08:55 EDT (History)
0 users

See Also:


Attachments
zip with source file and two sets of class files (21.29 KB, application/zip)
2005-12-14 11:50 EST, Wes Isberg CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Wes Isberg CLA 2005-12-14 11:49:02 EST
I'm noticing a VerifyError when building with AJDT but not on the command-line using the same version of AspectJ (current head).  

1) testToString(bugs.TimedCachedItemTest)java.lang.VerifyError: (class: bugs/Tim
edCachedItemTest$RandomToString, method: toString signature: ()Ljava/lang/String
;) Wrong return type in function
        at bugs.TimedCachedItemTest.testToString(TimedCachedItemTest.aj)

I mash the aspectj classes into the AJDT jars, so it's not unlikely that I caused this.  The code uses around advice and I seem to remember something with this - e.g., that if I specify some continue-after-error flag it will all work out, but I'm not remembering or seeing the flag at present.

I will attach the source along with classes built with the command-line compiler (in aspectj-src) and with AJDT (in bin).  Normally I'd isolate this a bit more, but don't have time and thought you might figure it out from the .class files.  My AJDT preferences (in the advanced and other tabs) are all deselected, except for (redundant?) no-line, so I'm doing full builds.
Comment 1 Wes Isberg CLA 2005-12-14 11:50:16 EST
Created attachment 31747 [details]
zip with source file and two sets of class files
Comment 2 Adrian Colyer CLA 2005-12-14 12:11:23 EST
how "current" is your "current head"? There have been quite a few changes going in over the last 24 hours to try and close things out for RC1 - so AJDT and HEAD are almost certainly different. The magic flag is -proceedOnError, but I don't think it will help in this case...
Comment 3 Wes Isberg CLA 2005-12-14 15:26:07 EST
I updated just before writing the bug, but I've been seeing the VerifyError for at least a few days.
Comment 4 Matt Chapman CLA 2005-12-16 06:50:15 EST
Wes,
I can't reproduce this with AJDT 1.3 RC1. Are you able to try in that environment?
Comment 5 Matt Chapman CLA 2006-04-13 08:55:43 EDT
Seems to be working ok for me. Reopen if you see the problem with a recent AJDT.