You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by se...@apache.org on 2015/08/21 22:39:18 UTC

svn commit: r1697056 - in /commons/proper/bcel/trunk/src/main/java/org/apache/commons/bcel6/verifier/structurals: ExecutionVisitor.java Frame.java InstConstraintVisitor.java Pass3bVerifier.java

Author: sebb
Date: Fri Aug 21 20:39:18 2015
New Revision: 1697056

URL: http://svn.apache.org/r1697056
Log:
Better name (_ not usual in method names)

Modified:
    commons/proper/bcel/trunk/src/main/java/org/apache/commons/bcel6/verifier/structurals/ExecutionVisitor.java
    commons/proper/bcel/trunk/src/main/java/org/apache/commons/bcel6/verifier/structurals/Frame.java
    commons/proper/bcel/trunk/src/main/java/org/apache/commons/bcel6/verifier/structurals/InstConstraintVisitor.java
    commons/proper/bcel/trunk/src/main/java/org/apache/commons/bcel6/verifier/structurals/Pass3bVerifier.java

Modified: commons/proper/bcel/trunk/src/main/java/org/apache/commons/bcel6/verifier/structurals/ExecutionVisitor.java
URL: http://svn.apache.org/viewvc/commons/proper/bcel/trunk/src/main/java/org/apache/commons/bcel6/verifier/structurals/ExecutionVisitor.java?rev=1697056&r1=1697055&r2=1697056&view=diff
==============================================================================
--- commons/proper/bcel/trunk/src/main/java/org/apache/commons/bcel6/verifier/structurals/ExecutionVisitor.java (original)
+++ commons/proper/bcel/trunk/src/main/java/org/apache/commons/bcel6/verifier/structurals/ExecutionVisitor.java Fri Aug 21 20:39:18 2015
@@ -838,7 +838,7 @@ public class ExecutionVisitor extends Em
     public void visitINVOKESPECIAL(INVOKESPECIAL o){
         if (o.getMethodName(cpg).equals(Constants.CONSTRUCTOR_NAME)){
             UninitializedObjectType t = (UninitializedObjectType) stack().peek(o.getArgumentTypes(cpg).length);
-            if (t == Frame.get_this()){
+            if (t == Frame.getThis()){
                 Frame._this = null;
             }
             stack().initializeObject(t);

Modified: commons/proper/bcel/trunk/src/main/java/org/apache/commons/bcel6/verifier/structurals/Frame.java
URL: http://svn.apache.org/viewvc/commons/proper/bcel/trunk/src/main/java/org/apache/commons/bcel6/verifier/structurals/Frame.java?rev=1697056&r1=1697055&r2=1697056&view=diff
==============================================================================
--- commons/proper/bcel/trunk/src/main/java/org/apache/commons/bcel6/verifier/structurals/Frame.java (original)
+++ commons/proper/bcel/trunk/src/main/java/org/apache/commons/bcel6/verifier/structurals/Frame.java Fri Aug 21 20:39:18 2015
@@ -125,15 +125,17 @@ public class Frame{
 
     /**
      * @return the _this
+     * @since 6.0
      */
-    public static UninitializedObjectType get_this() {
+    public static UninitializedObjectType getThis() {
         return _this;
     }
 
     /**
      * @param _this the _this to set
+     * @since 6.0
      */
-    public static void set_this(UninitializedObjectType _this) {
+    public static void setThis(UninitializedObjectType _this) {
         Frame._this = _this;
     }
 }

Modified: commons/proper/bcel/trunk/src/main/java/org/apache/commons/bcel6/verifier/structurals/InstConstraintVisitor.java
URL: http://svn.apache.org/viewvc/commons/proper/bcel/trunk/src/main/java/org/apache/commons/bcel6/verifier/structurals/InstConstraintVisitor.java?rev=1697056&r1=1697055&r2=1697056&view=diff
==============================================================================
--- commons/proper/bcel/trunk/src/main/java/org/apache/commons/bcel6/verifier/structurals/InstConstraintVisitor.java (original)
+++ commons/proper/bcel/trunk/src/main/java/org/apache/commons/bcel6/verifier/structurals/InstConstraintVisitor.java Fri Aug 21 20:39:18 2015
@@ -2882,7 +2882,7 @@ public class InstConstraintVisitor exten
     @Override
     public void visitRETURN(RETURN o){
         if (mg.getName().equals(Constants.CONSTRUCTOR_NAME)){// If we leave an <init> method
-            if ((Frame.get_this() != null) && (!(mg.getClassName().equals(Type.OBJECT.getClassName()))) ) {
+            if ((Frame.getThis() != null) && (!(mg.getClassName().equals(Type.OBJECT.getClassName()))) ) {
                 constraintViolated(o, "Leaving a constructor that itself did not call a constructor.");
             }
         }

Modified: commons/proper/bcel/trunk/src/main/java/org/apache/commons/bcel6/verifier/structurals/Pass3bVerifier.java
URL: http://svn.apache.org/viewvc/commons/proper/bcel/trunk/src/main/java/org/apache/commons/bcel6/verifier/structurals/Pass3bVerifier.java?rev=1697056&r1=1697055&r2=1697056&view=diff
==============================================================================
--- commons/proper/bcel/trunk/src/main/java/org/apache/commons/bcel6/verifier/structurals/Pass3bVerifier.java (original)
+++ commons/proper/bcel/trunk/src/main/java/org/apache/commons/bcel6/verifier/structurals/Pass3bVerifier.java Fri Aug 21 20:39:18 2015
@@ -350,7 +350,7 @@ public final class Pass3bVerifier extend
                 Frame f = new Frame(mg.getMaxLocals(),mg.getMaxStack());
                 if ( !mg.isStatic() ){
                     if (mg.getName().equals(Constants.CONSTRUCTOR_NAME)){
-                        Frame.set_this(new UninitializedObjectType(ObjectType.getInstance(jc.getClassName())));
+                        Frame.setThis(new UninitializedObjectType(ObjectType.getInstance(jc.getClassName())));
                         f.getLocals().set(0, Frame._this);
                     }
                     else{