Author: niallp Date: Thu Jan 13 00:04:49 2011 New Revision: 1058365 URL: http://svn.apache.org/viewvc?rev=1058365&view=rev Log: Use strip() instead of trim() - trim() doesn't remove all whitespace characters
Modified: commons/proper/lang/branches/LANG_2_X/src/main/java/org/apache/commons/lang/StringUtils.java commons/proper/lang/branches/LANG_2_X/src/test/java/org/apache/commons/lang/StringUtilsTest.java Modified: commons/proper/lang/branches/LANG_2_X/src/main/java/org/apache/commons/lang/StringUtils.java URL: http://svn.apache.org/viewvc/commons/proper/lang/branches/LANG_2_X/src/main/java/org/apache/commons/lang/StringUtils.java?rev=1058365&r1=1058364&r2=1058365&view=diff ============================================================================== --- commons/proper/lang/branches/LANG_2_X/src/main/java/org/apache/commons/lang/StringUtils.java (original) +++ commons/proper/lang/branches/LANG_2_X/src/main/java/org/apache/commons/lang/StringUtils.java Thu Jan 13 00:04:49 2011 @@ -6542,7 +6542,7 @@ public class StringUtils { * @since 2.6 */ public static String normalizeSpace(String str) { - str = trim(str); + str = strip(str); if(str == null || str.length() <= 2) { return str; } Modified: commons/proper/lang/branches/LANG_2_X/src/test/java/org/apache/commons/lang/StringUtilsTest.java URL: http://svn.apache.org/viewvc/commons/proper/lang/branches/LANG_2_X/src/test/java/org/apache/commons/lang/StringUtilsTest.java?rev=1058365&r1=1058364&r2=1058365&view=diff ============================================================================== --- commons/proper/lang/branches/LANG_2_X/src/test/java/org/apache/commons/lang/StringUtilsTest.java (original) +++ commons/proper/lang/branches/LANG_2_X/src/test/java/org/apache/commons/lang/StringUtilsTest.java Thu Jan 13 00:04:49 2011 @@ -1938,6 +1938,9 @@ public class StringUtilsTest extends Tes assertEquals("a", StringUtils.normalizeSpace(" a ")); assertEquals("a b c", StringUtils.normalizeSpace(" a b c ")); assertEquals("a b c", StringUtils.normalizeSpace("a\t\f\r b\u000B c\n")); + assertEquals("", StringUtils.normalizeSpace(WHITESPACE)); + assertEquals("A", StringUtils.normalizeSpace(WHITESPACE + " A " + WHITESPACE)); + assertEquals("A B", StringUtils.normalizeSpace(WHITESPACE + " A " + WHITESPACE + " B " + WHITESPACE)); } public void testLANG666() {