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:36:18 UTC
svn commit: r1606066 -
/commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/LocaleUtils.java
Author: ggregory
Date: Fri Jun 27 12:36:18 2014
New Revision: 1606066
URL: http://svn.apache.org/r1606066
Log:
Statement unnecessarily nested within else clause.
Modified:
commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/LocaleUtils.java
Modified: commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/LocaleUtils.java
URL: http://svn.apache.org/viewvc/commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/LocaleUtils.java?rev=1606066&r1=1606065&r2=1606066&view=diff
==============================================================================
--- commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/LocaleUtils.java (original)
+++ commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/LocaleUtils.java Fri Jun 27 12:36:18 2014
@@ -138,9 +138,8 @@ public class LocaleUtils {
(split[0].length() == 2 || split[0].length() == 3) &&
split[1].length() == 2 && StringUtils.isAllUpperCase(split[1])) {
return new Locale(split[0], split[1]);
- } else {
- throw new IllegalArgumentException("Invalid locale format: " + str);
}
+ throw new IllegalArgumentException("Invalid locale format: " + str);
case 2:
if (StringUtils.isAllLowerCase(split[0]) &&