[groovy] branch GROOVY_2_5_X updated: check input

[emilles]

This is an automated email from the ASF dual-hosted git repository.

emilles pushed a commit to branch GROOVY_2_5_X
in repository https://gitbox.apache.org/repos/asf/groovy.git


The following commit(s) were added to refs/heads/GROOVY_2_5_X by this push:
     new 12cfbb8e23 check input
12cfbb8e23 is described below

commit 12cfbb8e231c90b0c280a8505c691ea1eaa12ea7
Author: Eric Milles <eric.mil...@thomsonreuters.com>
AuthorDate: Mon Mar 13 15:19:09 2023 -0500

    check input
---
 .../java/org/codehaus/groovy/ast/expr/FieldExpression.java | 14 ++++++++------
 1 file changed, 8 insertions(+), 6 deletions(-)

diff --git a/src/main/java/org/codehaus/groovy/ast/expr/FieldExpression.java 
b/src/main/java/org/codehaus/groovy/ast/expr/FieldExpression.java
index 67350ff2e1..78a4c028a3 100644
--- a/src/main/java/org/codehaus/groovy/ast/expr/FieldExpression.java
+++ b/src/main/java/org/codehaus/groovy/ast/expr/FieldExpression.java
@@ -22,6 +22,8 @@ import org.codehaus.groovy.ast.ClassNode;
 import org.codehaus.groovy.ast.FieldNode;
 import org.codehaus.groovy.ast.GroovyCodeVisitor;
 
+import static java.util.Objects.requireNonNull;
+
 /**
  * Represents a field access such as the expression "this.foo".
  */
@@ -30,17 +32,17 @@ public class FieldExpression extends Expression {
     private final FieldNode field;
     private boolean useRef;
 
-    public FieldExpression(FieldNode field) {
-        this.field = field;
+    public FieldExpression(final FieldNode field) {
+        this.field = requireNonNull(field);
     }
 
     @Override
-    public void visit(GroovyCodeVisitor visitor) {
+    public void visit(final GroovyCodeVisitor visitor) {
         visitor.visitFieldExpression(this);
     }
 
     @Override
-    public Expression transformExpression(ExpressionTransformer transformer) {
+    public Expression transformExpression(final ExpressionTransformer 
transformer) {
         return this;
     }
 
@@ -63,7 +65,7 @@ public class FieldExpression extends Expression {
     }
 
     @Override
-    public void setType(ClassNode type) {
+    public void setType(final ClassNode type) {
         super.setType(type);
         field.setType(type);
     }
@@ -76,7 +78,7 @@ public class FieldExpression extends Expression {
         return useRef;
     }
 
-    public void setUseReferenceDirectly(boolean useRef) {
+    public void setUseReferenceDirectly(final boolean useRef) {
         this.useRef = useRef;
     }