You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@commons.apache.org by gg...@apache.org on 2005/08/12 07:31:27 UTC

svn commit: r232210 - /jakarta/commons/proper/lang/trunk/src/java/org/apache/commons/lang/StringUtils.java

Author: ggregory
Date: Thu Aug 11 22:31:25 2005
New Revision: 232210

URL: http://svn.apache.org/viewcvs?rev=232210&view=rev
Log:
Clean up of one method from Nathan Beyer [nbeyer@kc.rr.com]

Modified:
    jakarta/commons/proper/lang/trunk/src/java/org/apache/commons/lang/StringUtils.java

Modified: jakarta/commons/proper/lang/trunk/src/java/org/apache/commons/lang/StringUtils.java
URL: http://svn.apache.org/viewcvs/jakarta/commons/proper/lang/trunk/src/java/org/apache/commons/lang/StringUtils.java?rev=232210&r1=232209&r2=232210&view=diff
==============================================================================
--- jakarta/commons/proper/lang/trunk/src/java/org/apache/commons/lang/StringUtils.java (original)
+++ jakarta/commons/proper/lang/trunk/src/java/org/apache/commons/lang/StringUtils.java Thu Aug 11 22:31:25 2005
@@ -3210,11 +3210,7 @@
             if (str.charAt(lastIdx - 1) == CharUtils.CR) {
                 lastIdx--;
             }
-        } else if (last == CharUtils.CR) {
-            // why is this block empty?
-            // just to skip incrementing the index?
-          ;
-        } else {
+        } else if (last != CharUtils.CR) {
             lastIdx++;
         }
         return str.substring(0, lastIdx);



---------------------------------------------------------------------
To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: commons-dev-help@jakarta.apache.org