Bug 263323

Summary:	NPE building/weaving broken code
Product:	[Tools] AspectJ	Reporter:	Andrew Clement <aclement>
Component:	Compiler	Assignee:	aspectj inbox <aspectj-inbox>
Status:	RESOLVED FIXED	QA Contact:
Severity:	normal
Priority:	P3
Version:	DEVELOPMENT
Target Milestone:	1.6.4
Hardware:	PC
OS:	Windows NT
Whiteboard:

Description Andrew Clement

2009-02-02 19:38:05 EST

Create a class, create an aspect affecting that class via around advice.  Build.
Break the aspect, incremental build and the error comes out.  Then whitespace change in the class and save - this incremental build will break with:

 java.lang.NullPointerException at
   org.aspectj.weaver.bcel.BcelAdvice.canInline(BcelAdvice.java:215) at
   org.aspectj.weaver.bcel.BcelAdvice.implementOn(BcelAdvice.java:304) at
   org.aspectj.weaver.Shadow.implementMungers(Shadow.java:613) at 
   org.aspectj.weaver.Shadow.implement(Shadow.java:527) at
   org.aspectj.weaver.bcel.BcelClassWeaver.implement(BcelClassWeaver.java:3051)

Because the aspect isn't in good enough shape to answer questions

Comment 1 Andrew Clement

2009-02-03 12:35:05 EST

fixed - guards in place to cope with this situation and weaving doesn't proceed for the broken advice.