minor cleanups - add some final modifiers

Author: lukasstadler

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/nodes/PClosureFunctionRootNode.java

@@ -60,7 +60,7 @@ protected PClosureFunctionRootNode(PythonLanguage language, FrameDescriptor fram
         this.signature = signature;
     }
 
-    public String[] getCellVars() {
+    public final String[] getCellVars() {
         String[] cellVars = new String[cellVarSlots.length];
         for (int i = 0; i < cellVars.length; i++) {
             cellVars[i] = (String) cellVarSlots[i].getIdentifier();
@@ -69,7 +69,7 @@ public String[] getCellVars() {
     }
 
     @Override
-    public Signature getSignature() {
+    public final Signature getSignature() {
         return signature;
     }
 }

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/nodes/PClosureRootNode.java

@@ -46,6 +46,7 @@
 import com.oracle.graal.python.util.PythonUtils;
 import com.oracle.truffle.api.Assumption;
 import com.oracle.truffle.api.CompilerDirectives;
+import com.oracle.truffle.api.CompilerDirectives.CompilationFinal;
 import com.oracle.truffle.api.frame.Frame;
 import com.oracle.truffle.api.frame.FrameDescriptor;
 import com.oracle.truffle.api.frame.FrameSlot;
@@ -55,8 +56,8 @@
 public abstract class PClosureRootNode extends PRootNodeWithFileName {
     private final Assumption singleContextAssumption;
     private final boolean annotationsAvailable;
-    @CompilerDirectives.CompilationFinal(dimensions = 1) protected final FrameSlot[] freeVarSlots;
-    @CompilerDirectives.CompilationFinal(dimensions = 1) protected PCell[] closure;
+    @CompilationFinal(dimensions = 1) protected final FrameSlot[] freeVarSlots;
+    @CompilationFinal(dimensions = 1) protected PCell[] closure;
     private final int length;
 
     protected PClosureRootNode(PythonLanguage language, FrameDescriptor frameDescriptor, FrameSlot[] freeVarSlots, boolean hasAnnotations) {
@@ -67,7 +68,7 @@ protected PClosureRootNode(PythonLanguage language, FrameDescriptor frameDescrip
         this.annotationsAvailable = hasAnnotations;
     }
 
-    protected void addClosureCellsToLocals(Frame frame) {
+    protected final void addClosureCellsToLocals(Frame frame) {
         PCell[] frameClosure = PArguments.getClosure(frame);
         if (frameClosure != null) {
             if (singleContextAssumption.isValid() && closure == null) {
@@ -95,20 +96,20 @@ protected void addClosureCellsToLocals(Frame frame) {
         }
     }
 
-    protected void addClosureCellsToLocalsLoop(Frame frame, PCell[] frameClosure) {
+    protected final void addClosureCellsToLocalsLoop(Frame frame, PCell[] frameClosure) {
         for (int i = 0; i < length; i++) {
             frame.setObject(freeVarSlots[i], frameClosure[i]);
         }
     }
 
     @ExplodeLoop
-    protected void addClosureCellsToLocalsExploded(Frame frame, PCell[] frameClosure) {
+    protected final void addClosureCellsToLocalsExploded(Frame frame, PCell[] frameClosure) {
         for (int i = 0; i < length; i++) {
             frame.setObject(freeVarSlots[i], frameClosure[i]);
         }
     }
 
-    public boolean hasFreeVars() {
+    public final boolean hasFreeVars() {
         return freeVarSlots != null && freeVarSlots.length > 0;
     }