Revision: 10139
http://sourceforge.net/p/languagetool/code/10139
Author: dnaber
Date: 2013-05-23 21:06:33 +0000 (Thu, 23 May 2013)
Log Message:
-----------
small code cleanup
Modified Paths:
--------------
trunk/languagetool/languagetool-core/src/main/java/org/languagetool/rules/bitext/DifferentLengthRule.java
trunk/languagetool/languagetool-core/src/main/java/org/languagetool/rules/bitext/SameTranslationRule.java
trunk/languagetool/languagetool-core/src/main/java/org/languagetool/tools/ContextTools.java
Modified:
trunk/languagetool/languagetool-core/src/main/java/org/languagetool/rules/bitext/DifferentLengthRule.java
===================================================================
---
trunk/languagetool/languagetool-core/src/main/java/org/languagetool/rules/bitext/DifferentLengthRule.java
2013-05-23 20:04:14 UTC (rev 10138)
+++
trunk/languagetool/languagetool-core/src/main/java/org/languagetool/rules/bitext/DifferentLengthRule.java
2013-05-23 21:06:33 UTC (rev 10139)
@@ -29,7 +29,6 @@
* (smaller than 30% or longer by 250%).
*
* @author Marcin Miłkowski
- *
*/
public class DifferentLengthRule extends BitextRule {
@@ -49,7 +48,7 @@
@Override
public String getMessage() {
- return "Source and target translation lengths are very different!";
+ return "Source and target translation lengths are very different";
}
@Override
@@ -57,11 +56,9 @@
AnalyzedSentence targetText) throws IOException {
if (isLengthDifferent(getPureText(sourceText), getPureText(targetText))) {
- final RuleMatch[] ruleMatch = new RuleMatch[1];
final AnalyzedTokenReadings[] tokens = targetText.getTokens();
final int len = tokens[tokens.length - 1].getStartPos() +
tokens[tokens.length - 1].getToken().length();
- ruleMatch[0] = new RuleMatch(this, 1, len, getMessage());
- return ruleMatch;
+ return new RuleMatch[] { new RuleMatch(this, 1, len, getMessage()) };
}
return new RuleMatch[0];
}
Modified:
trunk/languagetool/languagetool-core/src/main/java/org/languagetool/rules/bitext/SameTranslationRule.java
===================================================================
---
trunk/languagetool/languagetool-core/src/main/java/org/languagetool/rules/bitext/SameTranslationRule.java
2013-05-23 20:04:14 UTC (rev 10138)
+++
trunk/languagetool/languagetool-core/src/main/java/org/languagetool/rules/bitext/SameTranslationRule.java
2013-05-23 21:06:33 UTC (rev 10139)
@@ -29,7 +29,6 @@
* is different.
*
* @author Marcin Miłkowski
- *
*/
public class SameTranslationRule extends BitextRule {
@@ -49,7 +48,7 @@
@Override
public String getMessage() {
- return "Source and target translation are the same!";
+ return "Source and target translation are the same";
}
@Override
@@ -59,11 +58,9 @@
//This is just heuristics, checking word count
if (sourceText.getTokensWithoutWhitespace().length > 3
&& getPureText(sourceText).equals(getPureText(targetText))) {
- final RuleMatch[] ruleMatch = new RuleMatch[1];
final AnalyzedTokenReadings[] tokens = targetText.getTokens();
final int len = tokens[tokens.length - 1].getStartPos() +
tokens[tokens.length - 1].getToken().length();
- ruleMatch[0] = new RuleMatch(this, 1, len, getMessage());
- return ruleMatch;
+ return new RuleMatch[] { new RuleMatch(this, 1, len, getMessage()) };
}
return new RuleMatch[0];
}
Modified:
trunk/languagetool/languagetool-core/src/main/java/org/languagetool/tools/ContextTools.java
===================================================================
---
trunk/languagetool/languagetool-core/src/main/java/org/languagetool/tools/ContextTools.java
2013-05-23 20:04:14 UTC (rev 10138)
+++
trunk/languagetool/languagetool-core/src/main/java/org/languagetool/tools/ContextTools.java
2013-05-23 21:06:33 UTC (rev 10139)
@@ -18,8 +18,6 @@
*/
package org.languagetool.tools;
-import org.languagetool.tools.StringTools;
-
/**
* Helper class to mark errors in text.
*/
@@ -46,14 +44,14 @@
startContent = 0;
}
int endContent = toPos + contextSize;
- final int fileLen = text.length();
- if (endContent > fileLen) {
+ final int textLength = text.length();
+ if (endContent > textLength) {
postfix = "";
- endContent = fileLen;
+ endContent = textLength;
}
// make "^" marker. inefficient but robust implementation:
final StringBuilder marker = new StringBuilder();
- final int totalLen = fileLen + prefix.length();
+ final int totalLen = textLength + prefix.length();
for (int i = 0; i < totalLen; i++) {
if (i >= fromPos && i < toPos) {
marker.append('^');
This was sent by the SourceForge.net collaborative development platform, the
world's largest Open Source development site.
------------------------------------------------------------------------------
Try New Relic Now & We'll Send You this Cool Shirt
New Relic is the only SaaS-based application performance monitoring service
that delivers powerful full stack analytics. Optimize and monitor your
browser, app, & servers with just a few lines of code. Try New Relic
and get this awesome Nerd Life shirt! http://p.sf.net/sfu/newrelic_d2d_may
_______________________________________________
Languagetool-commits mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/languagetool-commits
