[LanguageTool] SF.net SVN: languagetool:[7864] trunk/JLanguageTool/src

[dnaber]

Revision: 7864
          
http://languagetool.svn.sourceforge.net/languagetool/?rev=7864&view=rev
Author:   dnaber
Date:     2012-08-14 21:30:44 +0000 (Tue, 14 Aug 2012)
Log Message:
-----------
improve whitespace filtering by also removing \r - fixes one small TODO

Modified Paths:
--------------
    trunk/JLanguageTool/src/java/org/languagetool/tools/StringTools.java
    trunk/JLanguageTool/src/test/org/languagetool/tools/StringToolsTest.java

Modified: trunk/JLanguageTool/src/java/org/languagetool/tools/StringTools.java
===================================================================
--- trunk/JLanguageTool/src/java/org/languagetool/tools/StringTools.java        
2012-08-14 20:38:16 UTC (rev 7863)
+++ trunk/JLanguageTool/src/java/org/languagetool/tools/StringTools.java        
2012-08-14 21:30:44 UTC (rev 7864)
@@ -457,7 +457,7 @@
     final StringBuilder filter = new StringBuilder();
     for (int i = 0; i < str.length(); i++) {
       final char c = str.charAt(i);
-      if (c != '\n' && c != ' ' && c != '\t') {
+      if (c != '\n' && c != ' ' && c != '\t' && c != '\r') {
         filter.append(c);
       }
     }

Modified: 
trunk/JLanguageTool/src/test/org/languagetool/tools/StringToolsTest.java
===================================================================
--- trunk/JLanguageTool/src/test/org/languagetool/tools/StringToolsTest.java    
2012-08-14 20:38:16 UTC (rev 7863)
+++ trunk/JLanguageTool/src/test/org/languagetool/tools/StringToolsTest.java    
2012-08-14 21:30:44 UTC (rev 7864)
@@ -265,7 +265,7 @@
     assertEquals("", StringTools.trimWhitespace(""));
     assertEquals("", StringTools.trimWhitespace(" "));
     assertEquals("XXY", StringTools.trimWhitespace(" \nXX\t Y"));
-    // TODO: make this work assertEquals("XXY", StringTools.trimWhitespace(" 
\r\nXX\t Y"));
+    assertEquals("XXY", StringTools.trimWhitespace(" \r\nXX\t Y"));
     assertEquals("word", StringTools.trimWhitespace("word"));
   }
 

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
Languagetool-cvs@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/languagetool-cvs