I think that astore at the start of the exception block is the right pattern to use. I remember some bug about a decompiler choking on no store when it was expected.

1 files changed, 1 insertions, 0 deletions

diff --git a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/ast/AspectDeclaration.java b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/ast/AspectDeclaration.java
index 1039559cc..9203a43ab 100644
--- a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/ast/AspectDeclaration.java
+++ b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/ast/AspectDeclaration.java
@@ -749,6 +749,7 @@ public class AspectDeclaration extends TypeDeclaration {
 					exc.placeEnd();
 					codeStream.areturn();
 					exc.place();
+					codeStream.astore_1();
 					// this just returns null now - the old version used to throw the caught exception!
 					codeStream.aconst_null();
 					codeStream.areturn();