Return-Path: X-Original-To: archive-asf-public-internal@cust-asf2.ponee.io Delivered-To: archive-asf-public-internal@cust-asf2.ponee.io Received: from cust-asf.ponee.io (cust-asf.ponee.io [163.172.22.183]) by cust-asf2.ponee.io (Postfix) with ESMTP id 9B431200C35 for ; Sun, 12 Mar 2017 20:47:41 +0100 (CET) Received: by cust-asf.ponee.io (Postfix) id 99DB0160B8F; Sun, 12 Mar 2017 19:47:41 +0000 (UTC) Delivered-To: archive-asf-public@cust-asf.ponee.io Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by cust-asf.ponee.io (Postfix) with SMTP id BEBEE160B87 for ; Sun, 12 Mar 2017 20:47:40 +0100 (CET) Received: (qmail 65113 invoked by uid 500); 12 Mar 2017 19:47:33 -0000 Mailing-List: contact commits-help@commons.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@commons.apache.org Delivered-To: mailing list commits@commons.apache.org Received: (qmail 62431 invoked by uid 99); 12 Mar 2017 19:47:32 -0000 Received: from git1-us-west.apache.org (HELO git1-us-west.apache.org) (140.211.11.23) by apache.org (qpsmtpd/0.29) with ESMTP; Sun, 12 Mar 2017 19:47:32 +0000 Received: by git1-us-west.apache.org (ASF Mail Server at git1-us-west.apache.org, from userid 33) id 88246F4B68; Sun, 12 Mar 2017 19:47:31 +0000 (UTC) Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit From: mattsicker@apache.org To: commits@commons.apache.org Date: Sun, 12 Mar 2017 19:47:53 -0000 Message-Id: <12fc7e8484254ece8c1fe8167327d016@git.apache.org> In-Reply-To: References: X-Mailer: ASF-Git Admin Mailer Subject: [23/50] [abbrv] commons-pool git commit: Fix various IDE warnings archived-at: Sun, 12 Mar 2017 19:47:41 -0000 Fix various IDE warnings git-svn-id: https://svn.apache.org/repos/asf/commons/proper/pool/trunk@1735057 13f79535-47bb-0310-9956-ffa450edef68 Project: http://git-wip-us.apache.org/repos/asf/commons-pool/repo Commit: http://git-wip-us.apache.org/repos/asf/commons-pool/commit/2566fd6c Tree: http://git-wip-us.apache.org/repos/asf/commons-pool/tree/2566fd6c Diff: http://git-wip-us.apache.org/repos/asf/commons-pool/diff/2566fd6c Branch: refs/heads/master Commit: 2566fd6c479f86f46a0f0a65c4587572100b5cfc Parents: cd3f495 Author: Mark Thomas Authored: Tue Mar 15 11:16:13 2016 +0000 Committer: Mark Thomas Committed: Tue Mar 15 11:16:13 2016 +0000 ---------------------------------------------------------------------- src/main/java/org/apache/commons/pool2/BaseObject.java | 9 +++++++-- src/main/java/org/apache/commons/pool2/BaseObjectPool.java | 1 + .../apache/commons/pool2/impl/BaseGenericObjectPool.java | 7 ++++--- .../org/apache/commons/pool2/impl/BaseObjectPoolConfig.java | 1 + .../commons/pool2/impl/GenericKeyedObjectPoolConfig.java | 1 + .../apache/commons/pool2/impl/TestDefaultPooledObject.java | 4 +++- .../org/apache/commons/pool2/impl/TestPoolImplUtils.java | 2 ++ 7 files changed, 19 insertions(+), 6 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/commons-pool/blob/2566fd6c/src/main/java/org/apache/commons/pool2/BaseObject.java ---------------------------------------------------------------------- diff --git a/src/main/java/org/apache/commons/pool2/BaseObject.java b/src/main/java/org/apache/commons/pool2/BaseObject.java index 6a69abd..cc7f376 100644 --- a/src/main/java/org/apache/commons/pool2/BaseObject.java +++ b/src/main/java/org/apache/commons/pool2/BaseObject.java @@ -18,7 +18,7 @@ package org.apache.commons.pool2; /** * A base class for common functionality. - * + * * @since 2.4.3 */ public abstract class BaseObject { @@ -33,8 +33,13 @@ public abstract class BaseObject { return builder.toString(); } + /** + * Used by sub-classes to include the fields defined by the sub-class in the + * {@link #toString()} output. + * + * @param builder Field names and values are appended to this object + */ protected void toStringAppendFields(final StringBuilder builder) { // do nothing by default, needed for b/w compatibility. } - } http://git-wip-us.apache.org/repos/asf/commons-pool/blob/2566fd6c/src/main/java/org/apache/commons/pool2/BaseObjectPool.java ---------------------------------------------------------------------- diff --git a/src/main/java/org/apache/commons/pool2/BaseObjectPool.java b/src/main/java/org/apache/commons/pool2/BaseObjectPool.java index 2349622..7ef7187 100644 --- a/src/main/java/org/apache/commons/pool2/BaseObjectPool.java +++ b/src/main/java/org/apache/commons/pool2/BaseObjectPool.java @@ -119,6 +119,7 @@ public abstract class BaseObjectPool extends BaseObject implements ObjectPool private volatile boolean closed = false; + @Override protected void toStringAppendFields(final StringBuilder builder) { builder.append("closed="); builder.append(closed); http://git-wip-us.apache.org/repos/asf/commons-pool/blob/2566fd6c/src/main/java/org/apache/commons/pool2/impl/BaseGenericObjectPool.java ---------------------------------------------------------------------- diff --git a/src/main/java/org/apache/commons/pool2/impl/BaseGenericObjectPool.java b/src/main/java/org/apache/commons/pool2/impl/BaseGenericObjectPool.java index 6eafcd1..6e197c9 100644 --- a/src/main/java/org/apache/commons/pool2/impl/BaseGenericObjectPool.java +++ b/src/main/java/org/apache/commons/pool2/impl/BaseGenericObjectPool.java @@ -1176,7 +1176,7 @@ public abstract class BaseGenericObjectPool extends BaseObject { } } - + /** * Wrapper for objects under management by the pool. * @@ -1189,7 +1189,7 @@ public abstract class BaseGenericObjectPool extends BaseObject { static class IdentityWrapper { /** Wrapped object */ private final T instance; - + /** * Create a wrapper for an instance. * @@ -1209,7 +1209,7 @@ public abstract class BaseGenericObjectPool extends BaseObject { public boolean equals(final Object other) { return ((IdentityWrapper) other).instance == instance; } - + /** * @return the wrapped object */ @@ -1227,6 +1227,7 @@ public abstract class BaseGenericObjectPool extends BaseObject { } } + @Override protected void toStringAppendFields(final StringBuilder builder) { builder.append("maxTotal="); builder.append(maxTotal); http://git-wip-us.apache.org/repos/asf/commons-pool/blob/2566fd6c/src/main/java/org/apache/commons/pool2/impl/BaseObjectPoolConfig.java ---------------------------------------------------------------------- diff --git a/src/main/java/org/apache/commons/pool2/impl/BaseObjectPoolConfig.java b/src/main/java/org/apache/commons/pool2/impl/BaseObjectPoolConfig.java index 60b2384..2f1a595 100644 --- a/src/main/java/org/apache/commons/pool2/impl/BaseObjectPoolConfig.java +++ b/src/main/java/org/apache/commons/pool2/impl/BaseObjectPoolConfig.java @@ -640,6 +640,7 @@ public abstract class BaseObjectPoolConfig extends BaseObject implements Cloneab this.jmxNamePrefix = jmxNamePrefix; } + @Override protected void toStringAppendFields(final StringBuilder builder) { builder.append("lifo="); builder.append(lifo); http://git-wip-us.apache.org/repos/asf/commons-pool/blob/2566fd6c/src/main/java/org/apache/commons/pool2/impl/GenericKeyedObjectPoolConfig.java ---------------------------------------------------------------------- diff --git a/src/main/java/org/apache/commons/pool2/impl/GenericKeyedObjectPoolConfig.java b/src/main/java/org/apache/commons/pool2/impl/GenericKeyedObjectPoolConfig.java index bf05d53..2e201e5 100644 --- a/src/main/java/org/apache/commons/pool2/impl/GenericKeyedObjectPoolConfig.java +++ b/src/main/java/org/apache/commons/pool2/impl/GenericKeyedObjectPoolConfig.java @@ -182,6 +182,7 @@ public class GenericKeyedObjectPoolConfig extends BaseObjectPoolConfig { } } + @Override protected void toStringAppendFields(final StringBuilder builder) { super.toStringAppendFields(builder); builder.append(", minIdlePerKey="); http://git-wip-us.apache.org/repos/asf/commons-pool/blob/2566fd6c/src/test/java/org/apache/commons/pool2/impl/TestDefaultPooledObject.java ---------------------------------------------------------------------- diff --git a/src/test/java/org/apache/commons/pool2/impl/TestDefaultPooledObject.java b/src/test/java/org/apache/commons/pool2/impl/TestDefaultPooledObject.java index 07f1629..6484afd 100644 --- a/src/test/java/org/apache/commons/pool2/impl/TestDefaultPooledObject.java +++ b/src/test/java/org/apache/commons/pool2/impl/TestDefaultPooledObject.java @@ -27,7 +27,7 @@ import org.junit.Assert; import org.junit.Test; public class TestDefaultPooledObject { - + /** * JIRA: POOL-279 * @throws Exception @@ -39,6 +39,7 @@ public class TestDefaultPooledObject { final ExecutorService executor = Executors.newFixedThreadPool( Runtime.getRuntime().availableProcessors()*3); final Runnable allocateAndDeallocateTask = new Runnable() { + @Override public void run() { for (int i=0;i<10000;i++) { if (dpo.getIdleTimeMillis() < 0) { @@ -57,6 +58,7 @@ public class TestDefaultPooledObject { } }; final Runnable getIdleTimeTask = new Runnable() { + @Override public void run() { for (int i=0;i<10000;i++) { if (dpo.getIdleTimeMillis() < 0) { http://git-wip-us.apache.org/repos/asf/commons-pool/blob/2566fd6c/src/test/java/org/apache/commons/pool2/impl/TestPoolImplUtils.java ---------------------------------------------------------------------- diff --git a/src/test/java/org/apache/commons/pool2/impl/TestPoolImplUtils.java b/src/test/java/org/apache/commons/pool2/impl/TestPoolImplUtils.java index 6e0b20c..001b186 100644 --- a/src/test/java/org/apache/commons/pool2/impl/TestPoolImplUtils.java +++ b/src/test/java/org/apache/commons/pool2/impl/TestPoolImplUtils.java @@ -47,6 +47,7 @@ public class TestPoolImplUtils { } } + @SuppressWarnings("unused") private abstract static class FactoryAB extends BasePooledObjectFactory { } @@ -57,6 +58,7 @@ public class TestPoolImplUtils { private abstract static class FactoryC extends FactoryBA { } + @SuppressWarnings("unused") private abstract static class FactoryDE extends FactoryC{ }