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/04 23:46:54 UTC

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

Author: sebb
Date: Tue Jun  4 21:46:25 2013
New Revision: 1489639

URL: http://svn.apache.org/r1489639
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=1489639&r1=1489638&r2=1489639&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 21:46:25 2013
@@ -316,7 +316,7 @@ public class Threader {
 
             String subj = threadable.simplifiedSubject();
 
-            if (subj == null || subj == "") {
+            if (subj == null || subj.length() == 0) {
                 continue;
             }