This is an automated email from the ASF dual-hosted git repository.
sebb pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-collections.git
The following commit(s) were added to refs/heads/master by this push:
new cb57e04 COLLECTIONS-716 Don't include email address in Exception
messages
cb57e04 is described below
commit cb57e04fd6648ebdedb10b80c36fcf51a38f1b29
Author: Sebb <[email protected]>
AuthorDate: Mon Jun 17 23:49:07 2019 +0100
COLLECTIONS-716 Don't include email address in Exception messages
---
.../org/apache/commons/collections4/map/LRUMap.java | 19 +++++++------------
1 file changed, 7 insertions(+), 12 deletions(-)
diff --git a/src/main/java/org/apache/commons/collections4/map/LRUMap.java
b/src/main/java/org/apache/commons/collections4/map/LRUMap.java
index 7f411cb..ee13d39 100644
--- a/src/main/java/org/apache/commons/collections4/map/LRUMap.java
+++ b/src/main/java/org/apache/commons/collections4/map/LRUMap.java
@@ -271,8 +271,7 @@ public class LRUMap<K, V>
// remove
if(entry.before == null) {
throw new IllegalStateException("Entry.before is null." +
- " Please check that your keys are immutable, and that you
have used synchronization properly." +
- " If so, then please report this to
[email protected] as a bug.");
+ " This should not occur if your keys are immutable, and
you have used synchronization properly.");
}
entry.before.after = entry.after;
entry.after.before = entry.before;
@@ -283,7 +282,7 @@ public class LRUMap<K, V>
header.before = entry;
} else if (entry == header) {
throw new IllegalStateException("Can't move header to MRU" +
- " (please report this to [email protected])");
+ " 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(
"Entry.after=null, header.after=" + header.after + "
header.before=" + header.before +
" key=" + key + " value=" + value + " size=" + size +
" maxSize=" + maxSize +
- " Please check that your keys are immutable, and that
you have used synchronization properly." +
- " If so, then please report this to
[email protected] as a bug.");
+ " This should not occur if your keys are immutable,
and you have used synchronization properly.");
}
} else {
removeLRUEntry = removeLRU(reuse);
@@ -346,9 +344,8 @@ public class LRUMap<K, V>
throw new IllegalStateException(
"reuse=null, header.after=" + header.after + "
header.before=" + header.before +
" key=" + key + " value=" + value + " size=" + size +
" maxSize=" + maxSize +
- " Please check that your keys are immutable, and that
you have used synchronization properly." +
- " If so, then please report this to
[email protected] as a bug.");
- }
+ " This should not occur if your keys are immutable,
and you have used synchronization properly.");
+ }
reuseMapping(reuse, hashIndex, hashCode, key, value);
} else {
super.addMapping(hashIndex, hashCode, key, value);
@@ -387,8 +384,7 @@ public class LRUMap<K, V>
throw new IllegalStateException(
"Entry.next=null, data[removeIndex]=" + data[removeIndex]
+ " previous=" + previous +
" key=" + key + " value=" + value + " size=" + size + "
maxSize=" + maxSize +
- " Please check that your keys are immutable, and that you
have used synchronization properly." +
- " If so, then please report this to
[email protected] as a bug.");
+ " This should not occur if your keys are immutable, and
you have used synchronization properly.");
}
// reuse the entry
@@ -400,8 +396,7 @@ public class LRUMap<K, V>
throw new IllegalStateException(
"NPE, entry=" + entry + " entryIsHeader=" +
(entry==header) +
" key=" + key + " value=" + value + " size=" + size + "
maxSize=" + maxSize +
- " Please check that your keys are immutable, and that you
have used synchronization properly." +
- " If so, then please report this to
[email protected] as a bug.");
+ " This should not occur if your keys are immutable, and
you have used synchronization properly.");
}
}
