You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@commons.apache.org by se...@apache.org on 2016/04/30 15:33:09 UTC

svn commit: r1741765 - /commons/proper/net/trunk/src/main/java/org/apache/commons/net/nntp/Threader.java

Author: sebb
Date: Sat Apr 30 13:33:09 2016
New Revision: 1741765

URL: http://svn.apache.org/viewvc?rev=1741765&view=rev
Log:
Avoid Findbugs dead local store warning

Modified:
    commons/proper/net/trunk/src/main/java/org/apache/commons/net/nntp/Threader.java

Modified: commons/proper/net/trunk/src/main/java/org/apache/commons/net/nntp/Threader.java
URL: http://svn.apache.org/viewvc/commons/proper/net/trunk/src/main/java/org/apache/commons/net/nntp/Threader.java?rev=1741765&r1=1741764&r2=1741765&view=diff
==============================================================================
--- commons/proper/net/trunk/src/main/java/org/apache/commons/net/nntp/Threader.java (original)
+++ commons/proper/net/trunk/src/main/java/org/apache/commons/net/nntp/Threader.java Sat Apr 30 13:33:09 2016
@@ -109,7 +109,8 @@ public class Threader {
         // be a duplicate id, in which case we will need to generate a bogus placeholder id
         if (container != null) {
             if (container.threadable != null) { // oops! duplicate ids...
-                id = "<Bogus-id:" + (bogusIdCount++) + ">";
+                bogusIdCount++ ; // Avoid dead local store warning
+                id = "<Bogus-id:" + (bogusIdCount) + ">";
                 container = null;
             } else {
                 // The container just contained a forward reference to this message, so let's