This is an automated email from the ASF dual-hosted git repository.
garydgregory pushed a commit to branch POOL_2_X
in repository https://gitbox.apache.org/repos/asf/commons-pool.git
The following commit(s) were added to refs/heads/POOL_2_X by this push:
new 09cffdb0 Add messages when throwing NullPointerException.
09cffdb0 is described below
commit 09cffdb0830424722930593ddc2d9b3d2dfc3690
Author: Gary Gregory <[email protected]>
AuthorDate: Wed Jul 22 15:02:28 2026 -0400
Add messages when throwing NullPointerException.
---
.../org/apache/commons/pool2/impl/LinkedBlockingDeque.java | 10 +++++-----
src/main/java/org/apache/commons/pool2/impl/PoolImplUtils.java | 2 +-
2 files changed, 6 insertions(+), 6 deletions(-)
diff --git
a/src/main/java/org/apache/commons/pool2/impl/LinkedBlockingDeque.java
b/src/main/java/org/apache/commons/pool2/impl/LinkedBlockingDeque.java
index 056d5234..cfd873f5 100644
--- a/src/main/java/org/apache/commons/pool2/impl/LinkedBlockingDeque.java
+++ b/src/main/java/org/apache/commons/pool2/impl/LinkedBlockingDeque.java
@@ -347,17 +347,17 @@ final class LinkedBlockingDeque<E> extends
AbstractQueue<E>
* the given collection, added in traversal order of the
* collection's iterator.
*
- * @param c The collection of elements to initially contain
+ * @param coll The collection of elements to initially contain
* @throws NullPointerException if the specified collection or any
* of its elements are null
*/
- LinkedBlockingDeque(final Collection<? extends E> c) {
+ LinkedBlockingDeque(final Collection<? extends E> coll) {
this(Integer.MAX_VALUE);
lock.lock(); // Never contended, but necessary for visibility
try {
- for (final E e : c) {
- Objects.requireNonNull(e);
- if (!linkLast(e)) {
+ for (final E elt : coll) {
+ Objects.requireNonNull(elt, "elt");
+ if (!linkLast(elt)) {
throw new IllegalStateException("Deque full");
}
}
diff --git a/src/main/java/org/apache/commons/pool2/impl/PoolImplUtils.java
b/src/main/java/org/apache/commons/pool2/impl/PoolImplUtils.java
index a0f87118..9136c76f 100644
--- a/src/main/java/org/apache/commons/pool2/impl/PoolImplUtils.java
+++ b/src/main/java/org/apache/commons/pool2/impl/PoolImplUtils.java
@@ -196,7 +196,7 @@ final class PoolImplUtils {
*/
static ChronoUnit toChronoUnit(final TimeUnit timeUnit) {
// TODO when using Java >= 9: Use TimeUnit.toChronoUnit().
- switch (Objects.requireNonNull(timeUnit)) {
+ switch (Objects.requireNonNull(timeUnit, "timeUnit")) {
case NANOSECONDS:
return ChronoUnit.NANOS;
case MICROSECONDS:
