Skip to content

Commit 6677acc

Browse files
committed
Code clean ups.
git-svn-id: https://svn.apache.org/repos/asf/commons/proper/codec/trunk@1154427 13f79535-47bb-0310-9956-ffa450edef68
1 parent 1616779 commit 6677acc

4 files changed

Lines changed: 11 additions & 18 deletions

File tree

src/java/org/apache/commons/codec/language/bm/Lang.java

Lines changed: 2 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -199,11 +199,7 @@ private Lang(List<LangRule> rules, Languages languages) {
199199
*/
200200
public String guessLanguage(String text) {
201201
Languages.LanguageSet ls = guessLanguages(text);
202-
if (ls.isSingleton()) {
203-
return ls.getAny();
204-
} else {
205-
return Languages.ANY;
206-
}
202+
return ls.isSingleton() ? ls.getAny() : Languages.ANY;
207203
}
208204

209205
/**
@@ -235,10 +231,6 @@ public Languages.LanguageSet guessLanguages(String input) {
235231
}
236232

237233
Languages.LanguageSet ls = Languages.LanguageSet.from(langs);
238-
if (ls.equals(Languages.NO_LANGUAGES)) {
239-
return Languages.ANY_LANGUAGE;
240-
} else {
241-
return ls;
242-
}
234+
return ls.equals(Languages.NO_LANGUAGES) ? Languages.ANY_LANGUAGE : ls;
243235
}
244236
}

src/java/org/apache/commons/codec/language/bm/Languages.java

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -58,12 +58,9 @@ public class Languages {
5858
* A set of languages.
5959
*/
6060
public static abstract class LanguageSet {
61+
6162
public static LanguageSet from(Set<String> langs) {
62-
if (langs.isEmpty()) {
63-
return NO_LANGUAGES;
64-
} else {
65-
return new SomeLanguages(langs);
66-
}
63+
return langs.isEmpty() ? NO_LANGUAGES : new SomeLanguages(langs);
6764
}
6865

6966
public abstract boolean contains(String language);

src/java/org/apache/commons/codec/language/bm/NameType.java

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -24,10 +24,13 @@
2424
* @since 2.0
2525
*/
2626
public enum NameType {
27+
2728
/** Ashkenazi family names */
2829
ASHKENAZI("ash"),
30+
2931
/** Generic names and words */
3032
GENERIC("gen"),
33+
3134
/** Sephardic family names */
3235
SEPHARDIC("sep");
3336

src/java/org/apache/commons/codec/language/bm/PhoneticEngine.java

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -50,6 +50,7 @@
5050
* @since 2.0
5151
*/
5252
public class PhoneticEngine {
53+
5354
static class PhonemeBuilder {
5455

5556
public static PhonemeBuilder empty(Languages.LanguageSet languages) {
@@ -97,8 +98,9 @@ public String makeString() {
9798
// System.err.println(this.phonemes.getClass());
9899

99100
for (Rule.Phoneme ph : this.phonemes) {
100-
if (sb.length() > 0)
101+
if (sb.length() > 0) {
101102
sb.append("|");
103+
}
102104
sb.append(ph.getPhonemeText());
103105
}
104106

@@ -330,8 +332,7 @@ public String phoneticUtf8(String input, final Languages.LanguageSet languageSet
330332
return "(" + encode(remainder) + ")-(" + encode(combined) + ")";
331333
}
332334
// fixme: this case is invariant on l
333-
else if (input.length() >= 2 && input.substring(0, 2).equals("d'")) // check for d'
334-
{
335+
else if (input.length() >= 2 && input.substring(0, 2).equals("d'")) { // check for d'
335336
String remainder = input.substring(2);
336337
String combined = "d" + remainder;
337338
return "(" + encode(remainder) + ")-(" + encode(combined) + ")";

0 commit comments

Comments
 (0)