diff options
author | Gary D. Gregory <ggregory@apache.org> | 2011-08-02 18:04:51 +0000 |
---|---|---|
committer | Gary D. Gregory <ggregory@apache.org> | 2011-08-02 18:04:51 +0000 |
commit | ad5b9b01497834d0710eb9fa00a4372c3ceb60ec (patch) | |
tree | a47f5de26cad9377ce5a67e663f600a98edf1775 /src/main/java/org/apache/commons/lang3/CharUtils.java | |
parent | 9c28c3538d59431a6d05141b6117eb94c59999ec (diff) | |
download | apache-commons-lang-ad5b9b01497834d0710eb9fa00a4372c3ceb60ec.tar.gz |
[LANG-734] The CHAR_ARRAY cache in CharUtils duplicates the cache in java.lang.Character.
[LANG-735] Deprecate CharUtils.toCharacterObject(char) in favor of java.lang.Character.valueOf(char).
git-svn-id: https://svn.apache.org/repos/asf/commons/proper/lang/trunk@1153229 13f79535-47bb-0310-9956-ffa450edef68
Diffstat (limited to 'src/main/java/org/apache/commons/lang3/CharUtils.java')
-rw-r--r-- | src/main/java/org/apache/commons/lang3/CharUtils.java | 11 |
1 files changed, 4 insertions, 7 deletions
diff --git a/src/main/java/org/apache/commons/lang3/CharUtils.java b/src/main/java/org/apache/commons/lang3/CharUtils.java index 8008136e9..2241c0d32 100644 --- a/src/main/java/org/apache/commons/lang3/CharUtils.java +++ b/src/main/java/org/apache/commons/lang3/CharUtils.java @@ -48,7 +48,6 @@ public class CharUtils { "\u0078\u0079\u007a\u007b\u007c\u007d\u007e\u007f"; private static final String[] CHAR_STRING_ARRAY = new String[128]; - private static final Character[] CHAR_ARRAY = new Character[128]; /** * {@code \u000a} linefeed LF ('\n'). @@ -72,7 +71,6 @@ public class CharUtils { static { for (int i = 127; i >= 0; i--) { CHAR_STRING_ARRAY[i] = CHAR_STRING.substring(i, i + 1); - CHAR_ARRAY[i] = new Character((char) i); } } @@ -99,14 +97,13 @@ public class CharUtils { * CharUtils.toCharacterObject('A') = 'A' * </pre> * + * @deprecated Java 5 introduced {@link Character#valueOf(char)} which caches chars 0 through 127. * @param ch the character to convert * @return a Character of the specified character */ + @Deprecated public static Character toCharacterObject(char ch) { - if (ch < CHAR_ARRAY.length) { - return CHAR_ARRAY[ch]; - } - return new Character(ch); + return Character.valueOf(ch); } /** @@ -130,7 +127,7 @@ public class CharUtils { if (StringUtils.isEmpty(str)) { return null; } - return toCharacterObject(str.charAt(0)); + return Character.valueOf(str.charAt(0)); } //----------------------------------------------------------------------- |