garrettmills/commafeed
1
0
Fork 0
forked from Archives/Athou_commafeed
Code Pull Requests Activity

Merge pull request #641 from ebraminio/patch-1

Browse Source 
Use split limit
This commit is contained in:
Athou
2014-10-06 11:18:04 +02:00
parent 56bcc5ef5e 539d9c6d0e
commit 14ef5af936
1 changed files with 1 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
src/main/java/com/commafeed/backend/feed/EstimateDirection.java
View File

@@ -30,7 +30,7 @@ class EstimateDirection {
static boolean isRTL(String str) { static boolean isRTL(String str) {
int rtlCount = 0; int rtlCount = 0;
int total = 0; int total = 0;
String[] tokens = WORD_SEPARATOR_RE.split(str); String[] tokens = WORD_SEPARATOR_RE.split(str, 20); // limit splits to 20, usually enough
for (int i = 0; i < tokens.length; i++) { for (int i = 0; i < tokens.length; i++) {
String token = tokens[i]; String token = tokens[i];
if (startsWithRtl(token)) { if (startsWithRtl(token)) {
@@ -41,11 +41,6 @@ class EstimateDirection {
} else if (hasAnyLtr(token)) { } else if (hasAnyLtr(token)) {
total++; total++;
} }
// only checking 20 first words is usually enough
if (i == 20) {
break;
}
} }
return total == 0 ? false : ((float) rtlCount / total > RTL_DETECTION_THRESHOLD ? true : false); return total == 0 ? false : ((float) rtlCount / total > RTL_DETECTION_THRESHOLD ? true : false);