Revision: 7179
          
http://languagetool.svn.sourceforge.net/languagetool/?rev=7179&view=rev
Author:   milek_pl
Date:     2012-06-02 18:41:57 +0000 (Sat, 02 Jun 2012)
Log Message:
-----------
and fix some other bugs hidden in AnalyzedTokenReadings and AnalyzedToken

Modified Paths:
--------------
    trunk/JLanguageTool/src/java/org/languagetool/AnalyzedToken.java
    trunk/JLanguageTool/src/test/org/languagetool/AnalyzedTokenReadingsTest.java

Modified: trunk/JLanguageTool/src/java/org/languagetool/AnalyzedToken.java
===================================================================
--- trunk/JLanguageTool/src/java/org/languagetool/AnalyzedToken.java    
2012-06-02 18:31:56 UTC (rev 7178)
+++ trunk/JLanguageTool/src/java/org/languagetool/AnalyzedToken.java    
2012-06-02 18:41:57 UTC (rev 7179)
@@ -97,13 +97,13 @@
     }
     boolean found = true;
     if (!"".equals(an.getToken())) { //token cannot be null
-      found &= this.token.equals(an.getToken());      
+      found &= an.getToken().equals(this.token);      
     }
     if (an.getLemma() != null) {
-      found &= this.lemma.equals(an.getLemma());             
+      found &= an.getLemma().equals(this.lemma);             
     }
     if (an.getPOSTag() != null) {
-      found &= this.posTag.equals(an.getPOSTag());      
+      found &= an.getPOSTag().equals(this.posTag);      
     }
     return found;
   }

Modified: 
trunk/JLanguageTool/src/test/org/languagetool/AnalyzedTokenReadingsTest.java
===================================================================
--- 
trunk/JLanguageTool/src/test/org/languagetool/AnalyzedTokenReadingsTest.java    
    2012-06-02 18:31:56 UTC (rev 7178)
+++ 
trunk/JLanguageTool/src/test/org/languagetool/AnalyzedTokenReadingsTest.java    
    2012-06-02 18:41:57 UTC (rev 7179)
@@ -40,7 +40,7 @@
     assertEquals(false, tokenReadings.isParaEnd());
     tokenReadings.addReading(new AnalyzedToken("word", "PARA_END", null));
     assertEquals(true, tokenReadings.isParaEnd());
-    assertEquals(false, tokenReadings.isSentStart());
+    assertEquals(false, tokenReadings.isSentStart());            
     //but you can't add SENT_START to a non-empty token
     //and get isSentStart == true
     tokenReadings.addReading(new AnalyzedToken("word", "SENT_START", null));
@@ -54,6 +54,15 @@
     AnalyzedToken aTok3 = new AnalyzedToken("word", "POS", "lemma");
     aTok3.setWhitespaceBefore(true);
     assertEquals(aTok3, tokenReadings.getAnalyzedToken(0));
+    final AnalyzedTokenReadings testReadings = new 
AnalyzedTokenReadings(aTok3);
+    testReadings.removeReading(aTok3);
+    assertTrue(testReadings.getReadingsLength()==1);
+    assertEquals(testReadings.getToken(), "word");
+    assertTrue(!testReadings.hasPosTag("POS"));
+    //now what about removing something that does not belong to testReadings?
+    testReadings.leaveReading(aTok2);
+    assertEquals(testReadings.getToken(), "word");
+    assertTrue(!testReadings.hasPosTag("POS"));
   }
 
   public void testHasPosTag() {

This was sent by the SourceForge.net collaborative development platform, the 
world's largest Open Source development site.


------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and 
threat landscape has changed and how IT managers can respond. Discussions 
will include endpoint security, mobile security and the latest in malware 
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
_______________________________________________
Languagetool-cvs mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/languagetool-cvs

Reply via email to