Upstream/LuckPerms
1
0
Fork 0
mirror of https://github.com/LuckPerms/LuckPerms.git synced 2024-11-24 11:38:40 +01:00
Code Issues Projects Releases Wiki Activity

Don't intern strings in ContextSets (#797)

Browse Source
This commit is contained in:
Luck 2018-02-26 17:27:28 +00:00
parent aa4833ed74
commit a42d27f056
No known key found for this signature in database
GPG Key ID: EFA9B3EC5FD90F8B
1 changed files with 2 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
api/src/main/java/me/lucko/luckperms/api/context/AbstractContextSet.java
View File

@ -126,8 +126,7 @@ abstract class AbstractContextSet implements ContextSet {
if (stringIsEmpty(key)) { if (stringIsEmpty(key)) {
throw new IllegalArgumentException("key is (effectively) empty"); throw new IllegalArgumentException("key is (effectively) empty");
} }
return key.toLowerCase();
return key.toLowerCase().intern();
} }
static String sanitizeValue(String value) { static String sanitizeValue(String value) {
@ -135,8 +134,7 @@ abstract class AbstractContextSet implements ContextSet {
if (stringIsEmpty(value)) { if (stringIsEmpty(value)) {
throw new IllegalArgumentException("value is (effectively) empty"); throw new IllegalArgumentException("value is (effectively) empty");
} }
return value;
return value.intern();
} }
private static boolean stringIsEmpty(String s) { private static boolean stringIsEmpty(String s) {