You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@commons.apache.org by db...@apache.org on 2015/08/24 00:50:52 UTC

svn commit: r1697275 - /commons/proper/bcel/trunk/src/main/java/org/apache/commons/bcel6/generic/InstructionFactory.java

Author: dbrosius
Date: Sun Aug 23 22:50:52 2015
New Revision: 1697275

URL: http://svn.apache.org/r1697275
Log:
switch to more typical (idiomatic) java

Modified:
    commons/proper/bcel/trunk/src/main/java/org/apache/commons/bcel6/generic/InstructionFactory.java

Modified: commons/proper/bcel/trunk/src/main/java/org/apache/commons/bcel6/generic/InstructionFactory.java
URL: http://svn.apache.org/viewvc/commons/proper/bcel/trunk/src/main/java/org/apache/commons/bcel6/generic/InstructionFactory.java?rev=1697275&r1=1697274&r2=1697275&view=diff
==============================================================================
--- commons/proper/bcel/trunk/src/main/java/org/apache/commons/bcel6/generic/InstructionFactory.java (original)
+++ commons/proper/bcel/trunk/src/main/java/org/apache/commons/bcel6/generic/InstructionFactory.java Sun Aug 23 22:50:52 2015
@@ -409,7 +409,7 @@ public class InstructionFactory {
      * @param op operation, such as "+", "*", "&lt;&lt;", etc.
      */
     public static ArithmeticInstruction createBinaryOperation( String op, Type type ) {
-        char first = op.toCharArray()[0];
+        char first = op.charAt(0);
         switch (type.getType()) {
             case Constants.T_BYTE:
             case Constants.T_SHORT: