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
