You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by ma...@apache.org on 2016/12/05 11:32:41 UTC

svn commit: r1772642 - /tomcat/tc8.5.x/trunk/java/org/apache/catalina/storeconfig/StoreAppender.java

Author: markt
Date: Mon Dec  5 11:32:41 2016
New Revision: 1772642

URL: http://svn.apache.org/viewvc?rev=1772642&view=rev
Log:
FindBugs
pos doesn't need to be static.
It gets reset from indent at the beginning of every line and a StoreAppender instance is always responsible for an entire line so there is no need to share this between appender instances.

Modified:
    tomcat/tc8.5.x/trunk/java/org/apache/catalina/storeconfig/StoreAppender.java

Modified: tomcat/tc8.5.x/trunk/java/org/apache/catalina/storeconfig/StoreAppender.java
URL: http://svn.apache.org/viewvc/tomcat/tc8.5.x/trunk/java/org/apache/catalina/storeconfig/StoreAppender.java?rev=1772642&r1=1772641&r2=1772642&view=diff
==============================================================================
--- tomcat/tc8.5.x/trunk/java/org/apache/catalina/storeconfig/StoreAppender.java (original)
+++ tomcat/tc8.5.x/trunk/java/org/apache/catalina/storeconfig/StoreAppender.java Mon Dec  5 11:32:41 2016
@@ -41,7 +41,7 @@ public class StoreAppender {
             Float.class, Float.TYPE, Long.class, Long.TYPE, Short.class,
             Short.TYPE, InetAddress.class };
 
-    private static int pos = 0;
+    private int pos = 0;
 
     /**
      * Print the closing tag.



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org