You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pdfbox.apache.org by ti...@apache.org on 2021/06/15 18:21:58 UTC

svn commit: r1890818 - /pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/ttf/gsub/CompoundCharacterTokenizer.java

Author: tilman
Date: Tue Jun 15 18:21:58 2021
New Revision: 1890818

URL: http://svn.apache.org/viewvc?rev=1890818&view=rev
Log:
PDFBOX-4892: simplify, as suggested by valerybokov

Modified:
    pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/ttf/gsub/CompoundCharacterTokenizer.java

Modified: pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/ttf/gsub/CompoundCharacterTokenizer.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/ttf/gsub/CompoundCharacterTokenizer.java?rev=1890818&r1=1890817&r2=1890818&view=diff
==============================================================================
--- pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/ttf/gsub/CompoundCharacterTokenizer.java (original)
+++ pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/ttf/gsub/CompoundCharacterTokenizer.java Tue Jun 15 18:21:58 2021
@@ -33,7 +33,6 @@ import java.util.regex.Pattern;
  */
 public class CompoundCharacterTokenizer
 {
-
     private final Pattern regexExpression;
 
     public CompoundCharacterTokenizer(Set<String> compoundWords)
@@ -56,7 +55,6 @@ public class CompoundCharacterTokenizer
 
         while (regexMatcher.find())
         {
-
             int beginIndexOfNextMatch = regexMatcher.start();
 
             String prevToken = text.substring(lastIndexOfPrevMatch, beginIndexOfNextMatch);
@@ -74,7 +72,7 @@ public class CompoundCharacterTokenizer
 
         }
 
-        String tail = text.substring(lastIndexOfPrevMatch, text.length());
+        String tail = text.substring(lastIndexOfPrevMatch);
 
         if (tail.length() > 0)
         {
@@ -87,7 +85,7 @@ public class CompoundCharacterTokenizer
     private String getRegexFromTokens(Set<String> compoundWords)
     {
         StringJoiner sj = new StringJoiner(")|(", "(", ")");
-        compoundWords.stream().forEach(sj::add);
+        compoundWords.forEach(sj::add);
         return sj.toString();
     }