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

Posted to commits@commons.apache.org by gg...@apache.org on 2014/06/27 14:32:46 UTC

svn commit: r1606059 - /commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/text/translate/UnicodeUnpairedSurrogateRemover.java

Author: ggregory
Date: Fri Jun 27 12:32:45 2014
New Revision: 1606059

URL: http://svn.apache.org/r1606059
Log:
Statement unnecessarily nested within else clause.

Modified:
    commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/text/translate/UnicodeUnpairedSurrogateRemover.java

Modified: commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/text/translate/UnicodeUnpairedSurrogateRemover.java
URL: http://svn.apache.org/viewvc/commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/text/translate/UnicodeUnpairedSurrogateRemover.java?rev=1606059&r1=1606058&r2=1606059&view=diff
==============================================================================
--- commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/text/translate/UnicodeUnpairedSurrogateRemover.java (original)
+++ commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/text/translate/UnicodeUnpairedSurrogateRemover.java Fri Jun 27 12:32:45 2014
@@ -34,10 +34,9 @@ public class UnicodeUnpairedSurrogateRem
         if (codepoint >= Character.MIN_SURROGATE && codepoint <= Character.MAX_SURROGATE) {
             // It's a surrogate. Write nothing and say we've translated.
             return true;
-        } else {
-            // It's not a surrogate. Don't translate it.
-            return false;
         }
+        // It's not a surrogate. Don't translate it.
+        return false;
     }
 }