Revision: 9425
http://languagetool.svn.sourceforge.net/languagetool/?rev=9425&view=rev
Author: dnaber
Date: 2013-02-19 20:27:42 +0000 (Tue, 19 Feb 2013)
Log Message:
-----------
print warning if regex contains "||"
Modified Paths:
--------------
trunk/languagetool/languagetool-core/src/test/java/org/languagetool/rules/patterns/PatternRuleTest.java
Modified:
trunk/languagetool/languagetool-core/src/test/java/org/languagetool/rules/patterns/PatternRuleTest.java
===================================================================
---
trunk/languagetool/languagetool-core/src/test/java/org/languagetool/rules/patterns/PatternRuleTest.java
2013-02-19 18:01:30 UTC (rev 9424)
+++
trunk/languagetool/languagetool-core/src/test/java/org/languagetool/rules/patterns/PatternRuleTest.java
2013-02-19 20:27:42 UTC (rev 9425)
@@ -370,9 +370,7 @@
System.err.println("The " + lang.toString() + " rule: "
+ ruleId + ", token [" + tokenIndex + "], contains an empty string "
+ "\""
+ stringValue + "\" that is marked as regular expression.");
- } else if (isRegularExpression
- && !PROBABLE_PATTERN.matcher(stringValue)
- .find()) {
+ } else if (isRegularExpression &&
!PROBABLE_PATTERN.matcher(stringValue).find()) {
System.err.println("The " + lang.toString() + " rule: "
+ ruleId + ", token [" + tokenIndex + "], contains " + "\"" +
stringValue
+ "\" that is marked as regular expression but probably is not
one.");
@@ -395,6 +393,11 @@
+ "regular expression \".*\" which is useless: "
+ "(use an empty string without regexp=\"yes\" such as <token/>)");
}
+ if (isRegularExpression && stringValue.contains("||")) {
+ System.err.println("The " + lang.toString() + " rule: "
+ + ruleId + ", token [" + tokenIndex + "], marked as regular
expression contains "
+ + " '||', which is probably wrong");
+ }
if (isRegularExpression) {
final Matcher matcher = CHAR_SET_PATTERN.matcher(stringValue);
This was sent by the SourceForge.net collaborative development platform, the
world's largest Open Source development site.
------------------------------------------------------------------------------
Everyone hates slow websites. So do we.
Make your web apps faster with AppDynamics
Download AppDynamics Lite for free today:
http://p.sf.net/sfu/appdyn_d2d_feb
_______________________________________________
Languagetool-commits mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/languagetool-commits
