You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by ma...@apache.org on 2013/10/30 20:10:31 UTC
svn commit: r1537249 - in /commons/proper/pool/trunk:
findbugs-exclude-filter.xml
src/main/java/org/apache/commons/pool2/impl/AbandonedConfig.java
Author: markt
Date: Wed Oct 30 19:10:31 2013
New Revision: 1537249
URL: http://svn.apache.org/r1537249
Log:
Filter out the last remaining FindBugs warning that didn't appear until I tweaked by IDE settings. Expand that Javadoc comments to make the behaviour explicitly clear to users.
Modified:
commons/proper/pool/trunk/findbugs-exclude-filter.xml
commons/proper/pool/trunk/src/main/java/org/apache/commons/pool2/impl/AbandonedConfig.java
Modified: commons/proper/pool/trunk/findbugs-exclude-filter.xml
URL: http://svn.apache.org/viewvc/commons/proper/pool/trunk/findbugs-exclude-filter.xml?rev=1537249&r1=1537248&r2=1537249&view=diff
==============================================================================
--- commons/proper/pool/trunk/findbugs-exclude-filter.xml (original)
+++ commons/proper/pool/trunk/findbugs-exclude-filter.xml Wed Oct 30 19:10:31 2013
@@ -46,6 +46,12 @@
<Bug code="REC" />
</Match>
<Match>
+ <!-- Use of default system encoding is deliberate design choice -->
+ <Class name="org.apache.commons.pool2.impl.AbandonedConfig" />
+ <Method name="<init>" />
+ <Bug pattern="DM_DEFAULT_ENCODING " />
+ </Match>
+ <Match>
<!-- compareTo() method carries the appropriate warning -->
<Class name="org.apache.commons.pool2.impl.DefaultPooledObject" />
<Bug pattern="EQ_COMPARETO_USE_OBJECT_EQUALS " />
Modified: commons/proper/pool/trunk/src/main/java/org/apache/commons/pool2/impl/AbandonedConfig.java
URL: http://svn.apache.org/viewvc/commons/proper/pool/trunk/src/main/java/org/apache/commons/pool2/impl/AbandonedConfig.java?rev=1537249&r1=1537248&r2=1537249&view=diff
==============================================================================
--- commons/proper/pool/trunk/src/main/java/org/apache/commons/pool2/impl/AbandonedConfig.java (original)
+++ commons/proper/pool/trunk/src/main/java/org/apache/commons/pool2/impl/AbandonedConfig.java Wed Oct 30 19:10:31 2013
@@ -173,13 +173,14 @@ public class AbandonedConfig {
/**
* PrintWriter to use to log information on abandoned objects.
+ * Use of default system encoding is deliberate.
*/
private PrintWriter logWriter = new PrintWriter(System.out);
/**
* Returns the log writer being used by this configuration to log
* information on abandoned objects. If not set, a PrintWriter based on
- * System.out is used.
+ * System.out with the system default encoding is used.
*
* @return log writer in use
*/