Fix for pr103266: allow for nulls in the visitor (can happen if there are syntax errors)

1 files changed, 5 insertions, 0 deletions

diff --git a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/ast/MakeDeclsPublicVisitor.java b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/ast/MakeDeclsPublicVisitor.java
index b226e38dd..82fb8661b 100644
--- a/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/ast/MakeDeclsPublicVisitor.java
+++ b/org.aspectj.ajdt.core/src/org/aspectj/ajdt/internal/compiler/ast/MakeDeclsPublicVisitor.java
@@ -32,15 +32,18 @@ import org.aspectj.org.eclipse.jdt.internal.compiler.lookup.MethodScope;
 public class MakeDeclsPublicVisitor extends ASTVisitor {
 
 	public void endVisit(ConstructorDeclaration decl, ClassScope scope) {
+		if (decl.binding==null) return; 
 		decl.binding.modifiers = AstUtil.makePublic(decl.binding.modifiers);
 	}
 
 	public void endVisit(FieldDeclaration decl, MethodScope scope) {
+		if (decl.binding==null) return; 
 		decl.binding.modifiers = AstUtil.makePublic(decl.binding.modifiers);
 	}
 
 
 	public void endVisit(MethodDeclaration decl, ClassScope scope) {
+		if (decl.binding==null) return; 
 		decl.binding.modifiers = AstUtil.makePublic(decl.binding.modifiers);
 	}
 
@@ -50,6 +53,7 @@ public class MakeDeclsPublicVisitor extends ASTVisitor {
 	public void endVisit(
 		TypeDeclaration localTypeDeclaration,
 		BlockScope scope) {
+		if (localTypeDeclaration.binding==null) return; 
 		localTypeDeclaration.binding.modifiers = AstUtil.makePublic(localTypeDeclaration.binding.modifiers);
 	}
 
@@ -59,6 +63,7 @@ public class MakeDeclsPublicVisitor extends ASTVisitor {
 	public void endVisit(
 		TypeDeclaration memberTypeDeclaration,
 		ClassScope scope) {
+		if (memberTypeDeclaration.binding==null) return; 
 		memberTypeDeclaration.binding.modifiers = AstUtil.makePublic(memberTypeDeclaration.binding.modifiers);
 	}