Skip to content

Commit 8fa2bd1

Browse files
committed
GROOVY-11522: fix for NPE
1 parent 5e82e1e commit 8fa2bd1

1 file changed

Lines changed: 3 additions & 4 deletions

File tree

src/main/java/org/codehaus/groovy/classgen/VariableScopeVisitor.java

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -326,8 +326,7 @@ private void checkPropertyOnExplicitThis(final PropertyExpression expression) {
326326
String name = expression.getPropertyAsString();
327327
if (name == null || name.equals("class")) return;
328328
Variable member = findClassMember(currentClass, name);
329-
if (member == null) return;
330-
checkVariableContextAccess(member, expression);
329+
if (member != null) checkVariableContextAccess(member, expression);
331330
}
332331

333332
private void checkVariableContextAccess(final Variable variable, final Expression expression) {
@@ -552,9 +551,9 @@ public void visitDeclarationExpression(final DeclarationExpression expression) {
552551
@Override
553552
public void visitFieldExpression(final FieldExpression expression) {
554553
String name = expression.getFieldName();
555-
//TODO: change that to get the correct scope
554+
// TODO: change that to get the correct scope
556555
Variable variable = findVariableDeclaration(name);
557-
checkVariableContextAccess(variable, expression);
556+
if (variable != null) checkVariableContextAccess(variable, expression);
558557
}
559558

560559
@Override

0 commit comments

Comments
 (0)