Revision: 7992 http://languagetool.svn.sourceforge.net/languagetool/?rev=7992&view=rev Author: milek_pl Date: 2012-09-05 17:52:08 +0000 (Wed, 05 Sep 2012) Log Message: ----------- fix sf bug #3560925
Modified Paths: -------------- trunk/JLanguageTool/.classpath trunk/JLanguageTool/CHANGES.txt trunk/JLanguageTool/src/main/java/org/languagetool/rules/pl/PolishWordRepeatRule.java trunk/JLanguageTool/src/test/java/org/languagetool/rules/pl/PolishWordRepeatRuleTest.java Modified: trunk/JLanguageTool/.classpath =================================================================== --- trunk/JLanguageTool/.classpath 2012-09-05 09:47:56 UTC (rev 7991) +++ trunk/JLanguageTool/.classpath 2012-09-05 17:52:08 UTC (rev 7992) @@ -1,13 +1,9 @@ <?xml version="1.0" encoding="UTF-8"?> <classpath> - <classpathentry kind="src" path="src/test"/> - <classpathentry kind="src" path="src/dev"/> - <classpathentry kind="src" path="src/java"/> - <classpathentry excluding="test/|dev/|java/" kind="src" path="src"> - <attributes> - <attribute name="org.eclipse.jdt.launching.CLASSPATH_ATTR_LIBRARY_PATH_ENTRY" value="JLanguageTool/libs"/> - </attributes> - </classpathentry> + <classpathentry kind="src" path="src/main/java"/> + <classpathentry kind="src" path="src/main/resources"/> + <classpathentry kind="src" path="src/main/dev"/> + <classpathentry kind="src" path="src/test/java"/> <classpathentry kind="src" path="libs/native-lib"/> <classpathentry kind="lib" path="libs/build/junit.jar" sourcepath="C:/Users/user/.m2/repository/junit/junit/4.8.2/junit-4.8.2-sources.jar"> <attributes> Modified: trunk/JLanguageTool/CHANGES.txt =================================================================== --- trunk/JLanguageTool/CHANGES.txt 2012-09-05 09:47:56 UTC (rev 7991) +++ trunk/JLanguageTool/CHANGES.txt 2012-09-05 17:52:08 UTC (rev 7992) @@ -44,6 +44,9 @@ -Danish: -major update to the dictionary and the rules (Henrik Bendt) + + -Polish + -fixed bug with the word repetition rule (sf bug #3560925) -bugfix for command line: We removed XML from even plain text input. Now XML/HTML elements Modified: trunk/JLanguageTool/src/main/java/org/languagetool/rules/pl/PolishWordRepeatRule.java =================================================================== --- trunk/JLanguageTool/src/main/java/org/languagetool/rules/pl/PolishWordRepeatRule.java 2012-09-05 09:47:56 UTC (rev 7991) +++ trunk/JLanguageTool/src/main/java/org/languagetool/rules/pl/PolishWordRepeatRule.java 2012-09-05 17:52:08 UTC (rev 7992) @@ -97,6 +97,7 @@ boolean repetition = false; final TreeSet<String> inflectedWords = new TreeSet<String>(); String prevLemma, curLemma; + int curToken = 0; // start from real token, 0 = SENT_START for (int i = 1; i < tokens.length; i++) { final String token = tokens[i].getToken(); @@ -155,10 +156,11 @@ if (hasLemma) { curLemma = tokens[i].getAnalyzedToken(j).getLemma(); if (!prevLemma.equals(curLemma) && !notSentEnd) { - if (inflectedWords.contains(curLemma)) { + if (inflectedWords.contains(curLemma) && curToken != i) { repetition = true; } else { inflectedWords.add(tokens[i].getAnalyzedToken(j).getLemma()); + curToken = i; } } prevLemma = curLemma; Modified: trunk/JLanguageTool/src/test/java/org/languagetool/rules/pl/PolishWordRepeatRuleTest.java =================================================================== --- trunk/JLanguageTool/src/test/java/org/languagetool/rules/pl/PolishWordRepeatRuleTest.java 2012-09-05 09:47:56 UTC (rev 7991) +++ trunk/JLanguageTool/src/test/java/org/languagetool/rules/pl/PolishWordRepeatRuleTest.java 2012-09-05 17:52:08 UTC (rev 7992) @@ -40,8 +40,11 @@ matches = rule.match(langTool.getAnalyzedSentence("To jest zdanie próbne.")); assertEquals(0, matches.length); //repeated prepositions, don't count'em - matches = rule.match(langTool.getAnalyzedSentence("Na dyskotece tańczył jeszcze, choć był na bani.")); + matches = rule.match(langTool.getAnalyzedSentence("Na dyskotece tańczył jeszcze, choć był na bani.")); assertEquals(0, matches.length); + // sf bug report: + matches = rule.match(langTool.getAnalyzedSentence("Żadnych „ale”.")); + assertEquals(0, matches.length); //incorrect matches = rule.match(langTool.getAnalyzedSentence("Był on bowiem pięknym strzelcem bowiem.")); assertEquals(1, matches.length); 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