Apache
/
commons-collections
mirror of https://github.com/apache/commons-collections.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

COLLECTIONS-716 Don't include email address in Exception messages

This commit is contained in:
Sebb 2019-06-17 23:49:07 +01:00
parent ea9b4b51b0
commit cb57e04fd6
1 changed files with 7 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
src/main/java/org/apache/commons/collections4/map/LRUMap.java
View File

@ -271,8 +271,7 @@ public class LRUMap<K, V>
// remove // remove
if(entry.before == null) { if(entry.before == null) {
throw new IllegalStateException("Entry.before is null." + throw new IllegalStateException("Entry.before is null." +
" Please check that your keys are immutable, and that you have used synchronization properly." + " This should not occur if your keys are immutable, and you have used synchronization properly.");
" If so, then please report this to dev@commons.apache.org as a bug.");
} }
entry.before.after = entry.after; entry.before.after = entry.after;
entry.after.before = entry.before; entry.after.before = entry.before;
@ -283,7 +282,7 @@ public class LRUMap<K, V>
header.before = entry; header.before = entry;
} else if (entry == header) { } else if (entry == header) {
throw new IllegalStateException("Can't move header to MRU" + throw new IllegalStateException("Can't move header to MRU" +
" (please report this to dev@commons.apache.org)"); " This should not occur if your keys are immutable, and you have used synchronization properly.");
} }
} }
@ -334,8 +333,7 @@ public class LRUMap<K, V>
throw new IllegalStateException( throw new IllegalStateException(
"Entry.after=null, header.after=" + header.after + " header.before=" + header.before + "Entry.after=null, header.after=" + header.after + " header.before=" + header.before +
" key=" + key + " value=" + value + " size=" + size + " maxSize=" + maxSize + " key=" + key + " value=" + value + " size=" + size + " maxSize=" + maxSize +
" Please check that your keys are immutable, and that you have used synchronization properly." + " This should not occur if your keys are immutable, and you have used synchronization properly.");
" If so, then please report this to dev@commons.apache.org as a bug.");
} }
} else { } else {
removeLRUEntry = removeLRU(reuse); removeLRUEntry = removeLRU(reuse);
@ -346,9 +344,8 @@ public class LRUMap<K, V>
throw new IllegalStateException( throw new IllegalStateException(
"reuse=null, header.after=" + header.after + " header.before=" + header.before + "reuse=null, header.after=" + header.after + " header.before=" + header.before +
" key=" + key + " value=" + value + " size=" + size + " maxSize=" + maxSize + " key=" + key + " value=" + value + " size=" + size + " maxSize=" + maxSize +
" Please check that your keys are immutable, and that you have used synchronization properly." + " This should not occur if your keys are immutable, and you have used synchronization properly.");
" If so, then please report this to dev@commons.apache.org as a bug."); }
}
reuseMapping(reuse, hashIndex, hashCode, key, value); reuseMapping(reuse, hashIndex, hashCode, key, value);
} else { } else {
super.addMapping(hashIndex, hashCode, key, value); super.addMapping(hashIndex, hashCode, key, value);
@ -387,8 +384,7 @@ public class LRUMap<K, V>
throw new IllegalStateException( throw new IllegalStateException(
"Entry.next=null, data[removeIndex]=" + data[removeIndex] + " previous=" + previous + "Entry.next=null, data[removeIndex]=" + data[removeIndex] + " previous=" + previous +
" key=" + key + " value=" + value + " size=" + size + " maxSize=" + maxSize + " key=" + key + " value=" + value + " size=" + size + " maxSize=" + maxSize +
" Please check that your keys are immutable, and that you have used synchronization properly." + " This should not occur if your keys are immutable, and you have used synchronization properly.");
" If so, then please report this to dev@commons.apache.org as a bug.");
} }
// reuse the entry // reuse the entry
@ -400,8 +396,7 @@ public class LRUMap<K, V>
throw new IllegalStateException( throw new IllegalStateException(
"NPE, entry=" + entry + " entryIsHeader=" + (entry==header) + "NPE, entry=" + entry + " entryIsHeader=" + (entry==header) +
" key=" + key + " value=" + value + " size=" + size + " maxSize=" + maxSize + " key=" + key + " value=" + value + " size=" + size + " maxSize=" + maxSize +
" Please check that your keys are immutable, and that you have used synchronization properly." + " This should not occur if your keys are immutable, and you have used synchronization properly.");
" If so, then please report this to dev@commons.apache.org as a bug.");
} }
} }