jruby · Oct 12, 2014 · Oct 12, 2014
diff --git a/core/src/main/java/org/jruby/ir/instructions/BreakInstr.java b/core/src/main/java/org/jruby/ir/instructions/BreakInstr.java
@@ -92,10 +92,9 @@ public Instr clone(CloneInfo info) {
             }
 
             return new BreakInstr(returnValue.cloneForInlining(ii), scopeName);
+        } else {
+            throw new UnsupportedOperationException("Break instructions shouldn't show up outside closures.");
         }
-
-        throw new UnsupportedOperationException();
-        //return super.clone(ii);
     }
 
     @Override

diff --git a/core/src/main/java/org/jruby/ir/instructions/NonlocalReturnInstr.java b/core/src/main/java/org/jruby/ir/instructions/NonlocalReturnInstr.java
@@ -47,10 +47,9 @@ public Instr clone(CloneInfo info) {
             }
 
             return new NonlocalReturnInstr(returnValue.cloneForInlining(ii), methodName, maybeLambda);
+        } else {
+            throw new UnsupportedOperationException("Nonlocal returns shouldn't show up outside closures.");
         }
-
-        throw new UnsupportedOperationException();
-        //return super.clone(ii);
     }
 
     @Override

diff --git a/core/src/main/java/org/jruby/ir/instructions/ReceiveExceptionBase.java b/core/src/main/java/org/jruby/ir/instructions/ReceiveExceptionBase.java
@@ -5,7 +5,7 @@
 import org.jruby.ir.operands.Variable;
 import org.jruby.ir.transformations.inlining.CloneInfo;
 
-public class ReceiveExceptionBase extends Instr implements ResultInstr, FixedArityInstr {
+public abstract class ReceiveExceptionBase extends Instr implements ResultInstr, FixedArityInstr {
     protected Variable result;
 
     public ReceiveExceptionBase(Operation op, Variable result) {
@@ -34,10 +34,4 @@ public String toString() {
     public void updateResult(Variable v) {
         this.result = v;
     }
-
-    @Override
-    public Instr clone(CloneInfo info) {
-        throw new UnsupportedOperationException();
-    }
-
 }
diff --git a/core/src/main/java/org/jruby/ir/instructions/boxing/UnboxInstr.java b/core/src/main/java/org/jruby/ir/instructions/boxing/UnboxInstr.java
@@ -50,5 +50,4 @@ public String toString() {
     public Instr clone(CloneInfo info) {
         throw new UnsupportedOperationException();
     }
-
 }
diff --git a/core/src/main/java/org/jruby/ir/operands/LocalVariable.java b/core/src/main/java/org/jruby/ir/operands/LocalVariable.java
@@ -16,9 +16,9 @@
  * @author enebo
  */
 public class LocalVariable extends Variable implements DepthCloneable {
-    protected String name;
-    protected int scopeDepth;
-    protected int offset;
+    protected final String name;
+    protected final int scopeDepth;
+    protected final int offset;
 
     // FIXME: We should resolve to an index into an array but localvariable has no allocator
     public LocalVariable(String name, int scopeDepth, int location) {
-Original file line number
+Diff line change
@@ @@ -50,5 +50,4 @@ public String toString() { @@
         public Instr clone(CloneInfo info) {
             throw new UnsupportedOperationException();
         }
     }