Revision: 9915
          
http://languagetool.svn.sourceforge.net/languagetool/?rev=9915&view=rev
Author:   dnaber
Date:     2013-04-12 19:06:28 +0000 (Fri, 12 Apr 2013)
Log Message:
-----------
use constants if possible (not all languages have constants in Java's Locale 
class)

Modified Paths:
--------------
    
trunk/languagetool/languagetool-core/src/main/java/org/languagetool/rules/UppercaseSentenceStartRule.java
    
trunk/languagetool/languagetool-language-modules/de/src/main/java/org/languagetool/rules/de/GermanSpellerRule.java

Modified: 
trunk/languagetool/languagetool-core/src/main/java/org/languagetool/rules/UppercaseSentenceStartRule.java
===================================================================
--- 
trunk/languagetool/languagetool-core/src/main/java/org/languagetool/rules/UppercaseSentenceStartRule.java
   2013-04-12 17:16:25 UTC (rev 9914)
+++ 
trunk/languagetool/languagetool-core/src/main/java/org/languagetool/rules/UppercaseSentenceStartRule.java
   2013-04-12 19:06:28 UTC (rev 9915)
@@ -20,6 +20,7 @@
 
 import java.util.ArrayList;
 import java.util.List;
+import java.util.Locale;
 import java.util.ResourceBundle;
 
 import org.languagetool.AnalyzedSentence;
@@ -95,9 +96,9 @@
     boolean preventError = false;
     // TODO: why do only *these* languages have that special case?
     final String langCode = language.getShortName();
-    final boolean languageHasSpecialCases = langCode.equals("ru") || 
langCode.equals("pl") 
-            || langCode.equals("uk") || langCode.equals("be") || 
langCode.equals("en")
-            || langCode.equals("it") || langCode.equals("de");
+    final boolean languageHasSpecialCases = langCode.equals("ru") || 
langCode.equals("pl")
+            || langCode.equals("uk") || langCode.equals("be") || 
langCode.equals(Locale.ENGLISH.getLanguage())
+            || langCode.equals(Locale.ITALIAN.getLanguage()) || 
langCode.equals(Locale.GERMAN.getLanguage());
     if (languageHasSpecialCases) {
       //fix for lists; note - this will not always work for the last point in 
OOo,
       //as OOo might serve paragraphs in any order.

Modified: 
trunk/languagetool/languagetool-language-modules/de/src/main/java/org/languagetool/rules/de/GermanSpellerRule.java
===================================================================
--- 
trunk/languagetool/languagetool-language-modules/de/src/main/java/org/languagetool/rules/de/GermanSpellerRule.java
  2013-04-12 17:16:25 UTC (rev 9914)
+++ 
trunk/languagetool/languagetool-language-modules/de/src/main/java/org/languagetool/rules/de/GermanSpellerRule.java
  2013-04-12 19:06:28 UTC (rev 9915)
@@ -89,7 +89,7 @@
   }
 
   private static MorfologikSpeller getSpeller(Language language) {
-    if (!language.getShortName().equals("de")) {
+    if (!language.getShortName().equals(Locale.GERMAN.getLanguage())) {
       throw new RuntimeException("Language is not a variant of German: " + 
language);
     }
     try {

This was sent by the SourceForge.net collaborative development platform, the 
world's largest Open Source development site.


------------------------------------------------------------------------------
Precog is a next-generation analytics platform capable of advanced
analytics on semi-structured data. The platform includes APIs for building
apps and a phenomenal toolset for data science. Developers can use
our toolset for easy data analysis & visualization. Get a free account!
http://www2.precog.com/precogplatform/slashdotnewsletter
_______________________________________________
Languagetool-commits mailing list
Languagetool-commits@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/languagetool-commits

Reply via email to