Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 353457 - NPE when saving an aspect - Aspectj Internal Compiler Error
Summary: NPE when saving an aspect - Aspectj Internal Compiler Error
Status: RESOLVED FIXED
Alias: None
Product: AspectJ
Classification: Tools
Component: LTWeaving (show other bugs)
Version: 1.6.12   Edit
Hardware: Macintosh Mac OS X - Carbon (unsup.)
: P3 blocker (vote)
Target Milestone: 1.6.12   Edit
Assignee: aspectj inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
: 357067 (view as bug list)
Depends on:
Blocks:
 
Reported: 2011-07-30 17:23 EDT by Phil Hartley CLA
Modified: 2011-09-08 11:12 EDT (History)
3 users (show)

See Also:

Attachments
Two projects (14.12 KB, application/zip)
2011-07-30 17:23 EDT, Phil Hartley CLA 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Phil Hartley CLA 2011-07-30 17:23:36 EDT 
Created attachment 200625 [details]
Two projects

java.lang.NullPointerException
at org.aspectj.weaver.ShadowMunger.getBinaryFile(ShadowMunger.java:201)
at org.aspectj.weaver.ShadowMunger.getBinarySourceLocation(ShadowMunger.java:188)
at org.aspectj.weaver.model.AsmRelationshipProvider.createAdviceChild(AsmRelationshipProvider.java:699)
at org.aspectj.weaver.model.AsmRelationshipProvider.addChildNodes(AsmRelationshipProvider.java:657)
at org.aspectj.weaver.model.AsmRelationshipProvider.crea ... bleAspect$edu_uat_cs2011_observerpattern_Observable$observers(java.util.List)
end public class edu.uat.csc263.su10045.observermodel.DepartmentManager

I have attached a zip file containing the two projects that in combination caused this problem.  To recreate, load the projects, go into DepartmentManagerObservableAspect, modify the pointcut and save the file.
Comment 1 Andrew Clement CLA 2011-08-15 16:29:53 EDT 
this is now fixed.  It is due to the abstract aspect being only on the classpath.  
If you want to workaround it until an AJDT is available containing the change, just add the jar file (containing the abstract aspect) to the aspectpath rather than the classpath.
Comment 2 Andrew Clement CLA 2011-09-08 11:12:24 EDT 
*** Bug 357067 has been marked as a duplicate of this bug. ***