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 2013/06/05 00:15:09 UTC

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

Author: sebb
Date: Tue Jun  4 22:15:09 2013
New Revision: 1489655

URL: http://svn.apache.org/r1489655
Log:
Don't compare strings using ==

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=1489655&r1=1489654&r2=1489655&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 Tue Jun  4 22:15:09 2013
@@ -361,7 +361,7 @@ public class Threader {
             String subj = threadable.simplifiedSubject();
 
             // Dont thread together all subjectless messages
-            if (subj == null || subj == "") {
+            if (subj == null || subj.length() == 0) {
                 continue;
             }