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/06/20 08:21:59 UTC

svn commit: r1879025 - /pdfbox/branches/2.0/fontbox/src/main/java/org/apache/fontbox/ttf/GlyphSubstitutionTable.java

Author: tilman
Date: Sat Jun 20 08:21:59 2020
New Revision: 1879025

URL: http://svn.apache.org/viewvc?rev=1879025&view=rev
Log:
PDFBOX-4892: avoid "whitelist"

Modified:
    pdfbox/branches/2.0/fontbox/src/main/java/org/apache/fontbox/ttf/GlyphSubstitutionTable.java

Modified: pdfbox/branches/2.0/fontbox/src/main/java/org/apache/fontbox/ttf/GlyphSubstitutionTable.java
URL: http://svn.apache.org/viewvc/pdfbox/branches/2.0/fontbox/src/main/java/org/apache/fontbox/ttf/GlyphSubstitutionTable.java?rev=1879025&r1=1879024&r2=1879025&view=diff
==============================================================================
--- pdfbox/branches/2.0/fontbox/src/main/java/org/apache/fontbox/ttf/GlyphSubstitutionTable.java (original)
+++ pdfbox/branches/2.0/fontbox/src/main/java/org/apache/fontbox/ttf/GlyphSubstitutionTable.java Sat Jun 20 08:21:59 2020
@@ -405,7 +405,7 @@ public class GlyphSubstitutionTable exte
      *
      * @param langSysTables The {@code LangSysTable}s indicating {@code FeatureRecord}s to search
      * for
-     * @param enabledFeatures An optional whitelist of feature tags ({@code null} to allow all)
+     * @param enabledFeatures An optional list of feature tags ({@code null} to allow all)
      * @return The indicated {@code FeatureRecord}s
      */
     private List<FeatureRecord> getFeatureRecords(Collection<LangSysTable> langSysTables,
@@ -513,8 +513,8 @@ public class GlyphSubstitutionTable exte
 
     /**
      * Apply glyph substitutions to the supplied gid. The applicable substitutions are determined by
-     * the {@code scriptTags} which indicate the language of the gid, and by the
-     * {@code enabledFeatures} which acts as a whitelist.
+     * the {@code scriptTags} which indicate the language of the gid, and by the list of
+     * {@code enabledFeatures}.
      *
      * To ensure that a single gid isn't mapped to multiple substitutions, subsequent invocations
      * with the same gid will return the same result as the first, regardless of script or enabled
@@ -522,7 +522,7 @@ public class GlyphSubstitutionTable exte
      *
      * @param gid GID
      * @param scriptTags Script tags applicable to the gid (see {@link OpenTypeScript})
-     * @param enabledFeatures Whitelist of features to apply
+     * @param enabledFeatures list of features to apply
      */
     public int getSubstitution(int gid, String[] scriptTags, List<String> enabledFeatures)
     {