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/23 23:08:51 UTC
svn commit: r1697258 -
/commons/proper/bcel/trunk/src/main/java/org/apache/commons/bcel6/generic/BranchHandle.java
Author: sebb
Date: Sun Aug 23 21:08:51 2015
New Revision: 1697258
URL: http://svn.apache.org/r1697258
Log:
Document
Modified:
commons/proper/bcel/trunk/src/main/java/org/apache/commons/bcel6/generic/BranchHandle.java
Modified: commons/proper/bcel/trunk/src/main/java/org/apache/commons/bcel6/generic/BranchHandle.java
URL: http://svn.apache.org/viewvc/commons/proper/bcel/trunk/src/main/java/org/apache/commons/bcel6/generic/BranchHandle.java?rev=1697258&r1=1697257&r2=1697258&view=diff
==============================================================================
--- commons/proper/bcel/trunk/src/main/java/org/apache/commons/bcel6/generic/BranchHandle.java (original)
+++ commons/proper/bcel/trunk/src/main/java/org/apache/commons/bcel6/generic/BranchHandle.java Sun Aug 23 21:08:51 2015
@@ -76,7 +76,9 @@ public final class BranchHandle extends
@Override
void setPosition( int pos ) {
- i_position = getBI().setGetPosition(pos);
+ // Original code: i_position = bi.position = pos;
+ getBI().setPosition(pos);
+ i_position = pos;
}
@@ -115,7 +117,7 @@ public final class BranchHandle extends
/**
* Set new contents. Old instruction is disposed and may not be used anymore.
*/
- @Override
+ @Override // This is only done in order to apply the additional type check; could be merged with super impl.
public void setInstruction( Instruction i ) { // TODO could be package-protected?
super.setInstruction(i);
if (!(i instanceof BranchInstruction)) {