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 2017/10/14 15:16:50 UTC
svn commit: r1812189 - in /tomcat/trunk:
java/org/apache/tomcat/util/threads/TaskQueue.java
java/org/apache/tomcat/websocket/server/WsFilter.java
res/findbugs/filter-false-positives.xml
Author: markt
Date: Sat Oct 14 15:16:49 2017
New Revision: 1812189
URL: http://svn.apache.org/viewvc?rev=1812189&view=rev
Log:
Fix a handful of SpotBugs warnings
Modified:
tomcat/trunk/java/org/apache/tomcat/util/threads/TaskQueue.java
tomcat/trunk/java/org/apache/tomcat/websocket/server/WsFilter.java
tomcat/trunk/res/findbugs/filter-false-positives.xml
Modified: tomcat/trunk/java/org/apache/tomcat/util/threads/TaskQueue.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/util/threads/TaskQueue.java?rev=1812189&r1=1812188&r2=1812189&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/tomcat/util/threads/TaskQueue.java (original)
+++ tomcat/trunk/java/org/apache/tomcat/util/threads/TaskQueue.java Sat Oct 14 15:16:49 2017
@@ -32,7 +32,7 @@ public class TaskQueue extends LinkedBlo
private static final long serialVersionUID = 1L;
- private volatile ThreadPoolExecutor parent = null;
+ private transient volatile ThreadPoolExecutor parent = null;
// No need to be volatile. This is written and read in a single thread
// (when stopping a context and firing the listeners)
Modified: tomcat/trunk/java/org/apache/tomcat/websocket/server/WsFilter.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/websocket/server/WsFilter.java?rev=1812189&r1=1812188&r2=1812189&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/tomcat/websocket/server/WsFilter.java (original)
+++ tomcat/trunk/java/org/apache/tomcat/websocket/server/WsFilter.java Sat Oct 14 15:16:49 2017
@@ -33,7 +33,7 @@ public class WsFilter extends GenericFil
private static final long serialVersionUID = 1L;
- private WsServerContainer sc;
+ private transient WsServerContainer sc;
@Override
Modified: tomcat/trunk/res/findbugs/filter-false-positives.xml
URL: http://svn.apache.org/viewvc/tomcat/trunk/res/findbugs/filter-false-positives.xml?rev=1812189&r1=1812188&r2=1812189&view=diff
==============================================================================
--- tomcat/trunk/res/findbugs/filter-false-positives.xml (original)
+++ tomcat/trunk/res/findbugs/filter-false-positives.xml Sat Oct 14 15:16:49 2017
@@ -981,6 +981,11 @@
<Bug code="SF" />
</Match>
<Match>
+ <!-- No performance issue as there is no DNS resolution -->
+ <Class name="org.apache.tomcat.util.scan.StandardJarScanner" />
+ <Bug pattern="DMI_COLLECTION_OF_URLS" />
+ </Match>
+ <Match>
<!-- Yes the simple name is the same as the super class. Accept it. -->
<Class name="org.apache.tomcat.util.threads.ThreadPoolExecutor" />
<Bug code="Nm" />
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org