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