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 2020/05/05 16:51:47 UTC

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

Author: tilman
Date: Tue May  5 16:51:46 2020
New Revision: 1877404

URL: http://svn.apache.org/viewvc?rev=1877404&view=rev
Log:
PDFBOX-4071: use jdk8 functional syntax and new method 

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=1877404&r1=1877403&r2=1877404&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 May  5 16:51:46 2020
@@ -20,6 +20,7 @@ package org.apache.fontbox.ttf.gsub;
 import java.util.ArrayList;
 import java.util.List;
 import java.util.Set;
+import java.util.StringJoiner;
 import java.util.regex.Matcher;
 import java.util.regex.Pattern;
 
@@ -85,18 +86,9 @@ public class CompoundCharacterTokenizer
 
     private String getRegexFromTokens(Set<String> compoundWords)
     {
-        StringBuilder sb = new StringBuilder();
-
-        for (String compoundWord : compoundWords)
-        {
-            sb.append("(");
-            sb.append(compoundWord);
-            sb.append(")|");
-        }
-
-        sb.setLength(sb.length() - 1);
-
-        return sb.toString();
+        StringJoiner sj = new StringJoiner(")|(", "(", ")");
+        compoundWords.stream().forEach(sj::add);
+        return sj.toString();
     }
 
 }