[HB] Internally rename position() and substitute() to apply()
authorBehdad Esfahbod <behdad@behdad.org>
Mon, 18 May 2009 22:21:44 +0000 (18:21 -0400)
committerBehdad Esfahbod <behdad@behdad.org>
Mon, 2 Nov 2009 19:40:13 +0000 (14:40 -0500)
src/hb-ot-layout-gpos-private.h
src/hb-ot-layout-gsub-private.h

index a46f73d..834d360 100644 (file)
@@ -211,7 +211,7 @@ struct SinglePosFormat1 {
   friend struct SinglePos;
 
   private:
-  inline bool position (LOOKUP_ARGS_DEF) const {
+  inline bool apply (LOOKUP_ARGS_DEF) const {
     /* TODO */
     return false;
   }
@@ -234,7 +234,7 @@ struct SinglePosFormat2 {
   friend struct SinglePos;
 
   private:
-  inline bool position (LOOKUP_ARGS_DEF) const {
+  inline bool apply (LOOKUP_ARGS_DEF) const {
     /* TODO */
     return false;
   }
@@ -258,10 +258,10 @@ struct SinglePos {
 
   private:
 
-  inline bool position (LOOKUP_ARGS_DEF) const {
+  inline bool apply (LOOKUP_ARGS_DEF) const {
     switch (u.format) {
-    case 1: return u.format1->position (LOOKUP_ARGS);
-    case 2: return u.format2->position (LOOKUP_ARGS);
+    case 1: return u.format1->apply (LOOKUP_ARGS);
+    case 2: return u.format2->apply (LOOKUP_ARGS);
     default:return false;
     }
   }
@@ -297,7 +297,7 @@ struct PairPosFormat1 {
   friend struct PairPos;
 
   private:
-  inline bool position (LOOKUP_ARGS_DEF) const {
+  inline bool apply (LOOKUP_ARGS_DEF) const {
     /* TODO */
     return false;
   }
@@ -324,7 +324,7 @@ struct PairPosFormat2 {
   friend struct PairPos;
 
   private:
-  inline bool position (LOOKUP_ARGS_DEF) const {
+  inline bool apply (LOOKUP_ARGS_DEF) const {
     /* TODO */
     return false;
   }
@@ -364,10 +364,10 @@ struct PairPos {
 
   private:
 
-  inline bool position (LOOKUP_ARGS_DEF) const {
+  inline bool apply (LOOKUP_ARGS_DEF) const {
     switch (u.format) {
-    case 1: return u.format1->position (LOOKUP_ARGS);
-    case 2: return u.format2->position (LOOKUP_ARGS);
+    case 1: return u.format1->apply (LOOKUP_ARGS);
+    case 2: return u.format2->apply (LOOKUP_ARGS);
     default:return false;
     }
   }
@@ -402,7 +402,7 @@ struct CursivePosFormat1 {
   friend struct CursivePos;
 
   private:
-  inline bool position (LOOKUP_ARGS_DEF) const {
+  inline bool apply (LOOKUP_ARGS_DEF) const {
     /* TODO */
     return false;
   }
@@ -424,9 +424,9 @@ struct CursivePos {
 
   private:
 
-  inline bool position (LOOKUP_ARGS_DEF) const {
+  inline bool apply (LOOKUP_ARGS_DEF) const {
     switch (u.format) {
-    case 1: return u.format1->position (LOOKUP_ARGS);
+    case 1: return u.format1->apply (LOOKUP_ARGS);
     default:return false;
     }
   }
@@ -466,7 +466,7 @@ struct MarkBasePosFormat1 {
   friend struct MarkBasePos;
 
   private:
-  inline bool position (LOOKUP_ARGS_DEF) const {
+  inline bool apply (LOOKUP_ARGS_DEF) const {
     /* TODO */
     return false;
   }
@@ -492,9 +492,9 @@ struct MarkBasePos {
 
   private:
 
-  inline bool position (LOOKUP_ARGS_DEF) const {
+  inline bool apply (LOOKUP_ARGS_DEF) const {
     switch (u.format) {
-    case 1: return u.format1->position (LOOKUP_ARGS);
+    case 1: return u.format1->apply (LOOKUP_ARGS);
     default:return false;
     }
   }
@@ -551,7 +551,7 @@ struct MarkLigPosFormat1 {
   friend struct MarkLigPos;
 
   private:
-  inline bool position (LOOKUP_ARGS_DEF) const {
+  inline bool apply (LOOKUP_ARGS_DEF) const {
     /* TODO */
     return false;
   }
@@ -577,9 +577,9 @@ struct MarkLigPos {
 
   private:
 
-  inline bool position (LOOKUP_ARGS_DEF) const {
+  inline bool apply (LOOKUP_ARGS_DEF) const {
     switch (u.format) {
-    case 1: return u.format1->position (LOOKUP_ARGS);
+    case 1: return u.format1->apply (LOOKUP_ARGS);
     default:return false;
     }
   }
@@ -618,7 +618,7 @@ struct MarkMarkPosFormat1 {
   friend struct MarkMarkPos;
 
   private:
-  inline bool position (LOOKUP_ARGS_DEF) const {
+  inline bool apply (LOOKUP_ARGS_DEF) const {
     /* TODO */
     return false;
   }
@@ -644,9 +644,9 @@ struct MarkMarkPos {
 
   private:
 
-  inline bool position (LOOKUP_ARGS_DEF) const {
+  inline bool apply (LOOKUP_ARGS_DEF) const {
     switch (u.format) {
-    case 1: return u.format1->position (LOOKUP_ARGS);
+    case 1: return u.format1->apply (LOOKUP_ARGS);
     default:return false;
     }
   }
@@ -664,16 +664,16 @@ static inline bool position_lookup (LOOKUP_ARGS_DEF, unsigned int lookup_index);
 
 struct ContextPos : Context {
 
-  inline bool position (LOOKUP_ARGS_DEF) const {
-    return this->apply (LOOKUP_ARGS, position_lookup);
+  inline bool apply (LOOKUP_ARGS_DEF) const {
+    return Context::apply (LOOKUP_ARGS, position_lookup);
   }
 };
 ASSERT_SIZE (ContextPos, 2);
 
 struct ChainContextPos : ChainContext {
 
-  inline bool position (LOOKUP_ARGS_DEF) const {
-    return this->apply (LOOKUP_ARGS, position_lookup);
+  inline bool apply (LOOKUP_ARGS_DEF) const {
+    return ChainContext::apply (LOOKUP_ARGS, position_lookup);
   }
 };
 ASSERT_SIZE (ChainContextPos, 2);
@@ -686,7 +686,7 @@ struct ExtensionPosFormat1 {
   private:
   inline unsigned int get_type (void) const { return extensionLookupType; }
   inline unsigned int get_offset (void) const { return (extensionOffset[0] << 16) + extensionOffset[1]; }
-  inline bool position (LOOKUP_ARGS_DEF) const;
+  inline bool apply (LOOKUP_ARGS_DEF) const;
 
   private:
   USHORT       format;                 /* Format identifier. Set to 1. */
@@ -714,9 +714,9 @@ struct ExtensionPos {
     }
   }
 
-  inline bool position (LOOKUP_ARGS_DEF) const {
+  inline bool apply (LOOKUP_ARGS_DEF) const {
     switch (u.format) {
-    case 1: return u.format1->position (LOOKUP_ARGS);
+    case 1: return u.format1->apply (LOOKUP_ARGS);
     default:return false;
     }
   }
@@ -751,18 +751,18 @@ struct PosLookupSubTable {
 
   friend struct PosLookup;
 
-  inline bool position (LOOKUP_ARGS_DEF, unsigned int lookup_type) const {
+  inline bool apply (LOOKUP_ARGS_DEF, unsigned int lookup_type) const {
 
     switch (lookup_type) {
-    case GPOS_Single:                  return u.single->position (LOOKUP_ARGS);
-    case GPOS_Pair:                    return u.pair->position (LOOKUP_ARGS);
-    case GPOS_Cursive:                 return u.cursive->position (LOOKUP_ARGS);
-    case GPOS_MarkBase:                        return u.markBase->position (LOOKUP_ARGS);
-    case GPOS_MarkLig:                 return u.markLig->position (LOOKUP_ARGS);
-    case GPOS_MarkMark:                        return u.markMark->position (LOOKUP_ARGS);
-    case GPOS_Context:                 return u.context->position (LOOKUP_ARGS);
-    case GPOS_ChainContext:            return u.chainContext->position (LOOKUP_ARGS);
-    case GPOS_Extension:               return u.extension->position (LOOKUP_ARGS);
+    case GPOS_Single:                  return u.single->apply (LOOKUP_ARGS);
+    case GPOS_Pair:                    return u.pair->apply (LOOKUP_ARGS);
+    case GPOS_Cursive:                 return u.cursive->apply (LOOKUP_ARGS);
+    case GPOS_MarkBase:                        return u.markBase->apply (LOOKUP_ARGS);
+    case GPOS_MarkLig:                 return u.markLig->apply (LOOKUP_ARGS);
+    case GPOS_MarkMark:                        return u.markMark->apply (LOOKUP_ARGS);
+    case GPOS_Context:                 return u.context->apply (LOOKUP_ARGS);
+    case GPOS_ChainContext:            return u.chainContext->apply (LOOKUP_ARGS);
+    case GPOS_Extension:               return u.extension->apply (LOOKUP_ARGS);
     default:return false;
     }
   }
@@ -808,22 +808,22 @@ struct PosLookup : Lookup {
     return type;
   }
 
-  inline bool position_subtables (hb_ot_layout_t *layout,
-                                 hb_buffer_t    *buffer,
-                                 unsigned int    context_length,
-                                 unsigned int    nesting_level_left,
-                                 unsigned int    property) const {
+  inline bool apply_subtables (hb_ot_layout_t *layout,
+                              hb_buffer_t    *buffer,
+                              unsigned int    context_length,
+                              unsigned int    nesting_level_left,
+                              unsigned int    property) const {
     unsigned int lookup_type = get_type ();
     unsigned int lookup_flag = get_flag ();
 
     for (unsigned int i = 0; i < get_subtable_count (); i++)
-      if (get_subtable (i).position (LOOKUP_ARGS, lookup_type))
+      if (get_subtable (i).apply (LOOKUP_ARGS, lookup_type))
        return true;
 
     return false;
   }
 
-  inline bool position_once (hb_ot_layout_t *layout, hb_buffer_t *buffer) const {
+  inline bool apply_once (hb_ot_layout_t *layout, hb_buffer_t *buffer) const {
 
     unsigned int lookup_flag = get_flag ();
 
@@ -831,12 +831,12 @@ struct PosLookup : Lookup {
     if (!_hb_ot_layout_check_glyph_property (layout, IN_CURITEM (), lookup_flag, &property))
       return false;
 
-    return position_subtables (layout, buffer, NO_CONTEXT, MAX_NESTING_LEVEL, property);
+    return apply_subtables (layout, buffer, NO_CONTEXT, MAX_NESTING_LEVEL, property);
   }
 
-  bool position_string (hb_ot_layout_t *layout,
-                       hb_buffer_t    *buffer,
-                       hb_ot_layout_feature_mask_t mask) const {
+  bool apply_string (hb_ot_layout_t *layout,
+                    hb_buffer_t    *buffer,
+                    hb_ot_layout_feature_mask_t mask) const {
 
     bool ret = false;
 
@@ -850,7 +850,7 @@ struct PosLookup : Lookup {
 
       bool done;
       if (~IN_PROPERTIES (buffer->in_pos) & mask) {
-         done = position_once (layout, buffer);
+         done = apply_once (layout, buffer);
          ret |= done;
       } else {
           done = false;
@@ -887,7 +887,7 @@ struct GPOS : GSUBGPOS {
                               hb_buffer_t    *buffer,
                               unsigned int    lookup_index,
                               hb_ot_layout_feature_mask_t  mask) const {
-    return get_lookup (lookup_index).position_string (layout, buffer, mask);
+    return get_lookup (lookup_index).apply_string (layout, buffer, mask);
   }
 
 };
@@ -896,13 +896,13 @@ ASSERT_SIZE (GPOS, 10);
 
 /* Out-of-class implementation for methods recursing */
 
-inline bool ExtensionPosFormat1::position (LOOKUP_ARGS_DEF) const {
+inline bool ExtensionPosFormat1::apply (LOOKUP_ARGS_DEF) const {
   unsigned int lookup_type = get_type ();
 
   if (HB_UNLIKELY (lookup_type ==  GPOS_Extension))
     return false;
 
-  return ((PosLookupSubTable&)*(((char *) this) + get_offset ())).position (LOOKUP_ARGS, lookup_type);
+  return ((PosLookupSubTable&)*(((char *) this) + get_offset ())).apply (LOOKUP_ARGS, lookup_type);
 }
 
 static inline bool position_lookup (LOOKUP_ARGS_DEF, unsigned int lookup_index) {
@@ -916,7 +916,7 @@ static inline bool position_lookup (LOOKUP_ARGS_DEF, unsigned int lookup_index)
   if (HB_UNLIKELY (context_length < 1))
     return false;
 
-  return l.position_subtables (layout, buffer, context_length, nesting_level_left, property);
+  return l.apply_subtables (layout, buffer, context_length, nesting_level_left, property);
 }
 
 
index e1d17c8..bf2381e 100644 (file)
@@ -111,7 +111,7 @@ struct SingleSubst {
     }
   }
 
-  inline bool substitute (LOOKUP_ARGS_DEF) const {
+  inline bool apply (LOOKUP_ARGS_DEF) const {
 
     hb_codepoint_t glyph_id = IN_CURGLYPH ();
 
@@ -152,7 +152,7 @@ struct Sequence {
       _hb_ot_layout_set_glyph_property (layout, substitute[n], property);
   }
 
-  inline bool substitute_sequence (LOOKUP_ARGS_DEF) const {
+  inline bool apply (LOOKUP_ARGS_DEF) const {
 
     if (HB_UNLIKELY (!substitute.len))
       return false;
@@ -186,13 +186,13 @@ struct MultipleSubstFormat1 {
 
   private:
 
-  inline bool substitute (LOOKUP_ARGS_DEF) const {
+  inline bool apply (LOOKUP_ARGS_DEF) const {
 
     unsigned int index = (this+coverage) (IN_CURGLYPH ());
     if (HB_LIKELY (index == NOT_COVERED))
       return false;
 
-    return (this+sequence[index]).substitute_sequence (LOOKUP_ARGS);
+    return (this+sequence[index]).apply (LOOKUP_ARGS);
   }
 
   private:
@@ -212,9 +212,9 @@ struct MultipleSubst {
 
   private:
 
-  inline bool substitute (LOOKUP_ARGS_DEF) const {
+  inline bool apply (LOOKUP_ARGS_DEF) const {
     switch (u.format) {
-    case 1: return u.format1->substitute (LOOKUP_ARGS);
+    case 1: return u.format1->apply (LOOKUP_ARGS);
     default:return false;
     }
   }
@@ -238,7 +238,7 @@ struct AlternateSubstFormat1 {
 
   private:
 
-  inline bool substitute (LOOKUP_ARGS_DEF) const {
+  inline bool apply (LOOKUP_ARGS_DEF) const {
 
     hb_codepoint_t glyph_id = IN_CURGLYPH ();
 
@@ -293,9 +293,9 @@ struct AlternateSubst {
 
   private:
 
-  inline bool substitute (LOOKUP_ARGS_DEF) const {
+  inline bool apply (LOOKUP_ARGS_DEF) const {
     switch (u.format) {
-    case 1: return u.format1->substitute (LOOKUP_ARGS);
+    case 1: return u.format1->apply (LOOKUP_ARGS);
     default:return false;
     }
   }
@@ -314,7 +314,7 @@ struct Ligature {
   friend struct LigatureSet;
 
   private:
-  inline bool substitute (LOOKUP_ARGS_DEF, bool is_mark) const {
+  inline bool apply (LOOKUP_ARGS_DEF, bool is_mark) const {
 
     unsigned int i, j;
     unsigned int count = component.len;
@@ -393,12 +393,12 @@ struct LigatureSet {
 
   private:
 
-  inline bool substitute (LOOKUP_ARGS_DEF, bool is_mark) const {
+  inline bool apply (LOOKUP_ARGS_DEF, bool is_mark) const {
 
     unsigned int num_ligs = ligature.len;
     for (unsigned int i = 0; i < num_ligs; i++) {
       const Ligature &lig = this+ligature[i];
-      if (lig.substitute (LOOKUP_ARGS, is_mark))
+      if (lig.apply (LOOKUP_ARGS, is_mark))
         return true;
     }
 
@@ -418,7 +418,7 @@ struct LigatureSubstFormat1 {
 
   private:
 
-  inline bool substitute (LOOKUP_ARGS_DEF) const {
+  inline bool apply (LOOKUP_ARGS_DEF) const {
 
     hb_codepoint_t glyph_id = IN_CURGLYPH ();
 
@@ -430,7 +430,7 @@ struct LigatureSubstFormat1 {
       return false;
 
     const LigatureSet &lig_set = this+ligatureSet[index];
-    return lig_set.substitute (LOOKUP_ARGS, first_is_mark);
+    return lig_set.apply (LOOKUP_ARGS, first_is_mark);
   }
 
   private:
@@ -450,9 +450,9 @@ struct LigatureSubst {
 
   private:
 
-  inline bool substitute (LOOKUP_ARGS_DEF) const {
+  inline bool apply (LOOKUP_ARGS_DEF) const {
     switch (u.format) {
-    case 1: return u.format1->substitute (LOOKUP_ARGS);
+    case 1: return u.format1->apply (LOOKUP_ARGS);
     default:return false;
     }
   }
@@ -471,16 +471,16 @@ static inline bool substitute_lookup (LOOKUP_ARGS_DEF, unsigned int lookup_index
 
 struct ContextSubst : Context {
 
-  inline bool substitute (LOOKUP_ARGS_DEF) const {
-    return this->apply (LOOKUP_ARGS, substitute_lookup);
+  inline bool apply (LOOKUP_ARGS_DEF) const {
+    return Context::apply (LOOKUP_ARGS, substitute_lookup);
   }
 };
 ASSERT_SIZE (ContextSubst, 2);
 
 struct ChainContextSubst : ChainContext {
 
-  inline bool substitute (LOOKUP_ARGS_DEF) const {
-    return this->apply (LOOKUP_ARGS, substitute_lookup);
+  inline bool apply (LOOKUP_ARGS_DEF) const {
+    return ChainContext::apply (LOOKUP_ARGS, substitute_lookup);
   }
 };
 ASSERT_SIZE (ChainContextSubst, 2);
@@ -493,7 +493,7 @@ struct ExtensionSubstFormat1 {
   private:
   inline unsigned int get_type (void) const { return extensionLookupType; }
   inline unsigned int get_offset (void) const { return (extensionOffset[0] << 16) + extensionOffset[1]; }
-  inline bool substitute (LOOKUP_ARGS_DEF) const;
+  inline bool apply (LOOKUP_ARGS_DEF) const;
 
   private:
   USHORT       format;                 /* Format identifier. Set to 1. */
@@ -521,9 +521,9 @@ struct ExtensionSubst {
     }
   }
 
-  inline bool substitute (LOOKUP_ARGS_DEF) const {
+  inline bool apply (LOOKUP_ARGS_DEF) const {
     switch (u.format) {
-    case 1: return u.format1->substitute (LOOKUP_ARGS);
+    case 1: return u.format1->apply (LOOKUP_ARGS);
     default:return false;
     }
   }
@@ -542,7 +542,7 @@ struct ReverseChainSingleSubstFormat1 {
   friend struct ReverseChainSingleSubst;
 
   private:
-  inline bool substitute (LOOKUP_ARGS_DEF) const {
+  inline bool apply (LOOKUP_ARGS_DEF) const {
 
     if (HB_UNLIKELY (context_length != NO_CONTEXT))
       return false; /* No chaining to this type */
@@ -597,9 +597,9 @@ struct ReverseChainSingleSubst {
 
   private:
 
-  inline bool substitute (LOOKUP_ARGS_DEF) const {
+  inline bool apply (LOOKUP_ARGS_DEF) const {
     switch (u.format) {
-    case 1: return u.format1->substitute (LOOKUP_ARGS);
+    case 1: return u.format1->apply (LOOKUP_ARGS);
     default:return false;
     }
   }
@@ -633,17 +633,17 @@ struct SubstLookupSubTable {
 
   friend struct SubstLookup;
 
-  inline bool substitute (LOOKUP_ARGS_DEF, unsigned int lookup_type) const {
+  inline bool apply (LOOKUP_ARGS_DEF, unsigned int lookup_type) const {
 
     switch (lookup_type) {
-    case GSUB_Single:                  return u.single->substitute (LOOKUP_ARGS);
-    case GSUB_Multiple:                        return u.multiple->substitute (LOOKUP_ARGS);
-    case GSUB_Alternate:               return u.alternate->substitute (LOOKUP_ARGS);
-    case GSUB_Ligature:                        return u.ligature->substitute (LOOKUP_ARGS);
-    case GSUB_Context:                 return u.context->substitute (LOOKUP_ARGS);
-    case GSUB_ChainContext:            return u.chainContext->substitute (LOOKUP_ARGS);
-    case GSUB_Extension:               return u.extension->substitute (LOOKUP_ARGS);
-    case GSUB_ReverseChainSingle:      return u.reverseChainContextSingle->substitute (LOOKUP_ARGS);
+    case GSUB_Single:                  return u.single->apply (LOOKUP_ARGS);
+    case GSUB_Multiple:                        return u.multiple->apply (LOOKUP_ARGS);
+    case GSUB_Alternate:               return u.alternate->apply (LOOKUP_ARGS);
+    case GSUB_Ligature:                        return u.ligature->apply (LOOKUP_ARGS);
+    case GSUB_Context:                 return u.context->apply (LOOKUP_ARGS);
+    case GSUB_ChainContext:            return u.chainContext->apply (LOOKUP_ARGS);
+    case GSUB_Extension:               return u.extension->apply (LOOKUP_ARGS);
+    case GSUB_ReverseChainSingle:      return u.reverseChainContextSingle->apply (LOOKUP_ARGS);
     default:return false;
     }
   }
@@ -695,22 +695,22 @@ struct SubstLookup : Lookup {
     }
   }
 
-  inline bool substitute_subtables (hb_ot_layout_t *layout,
-                                   hb_buffer_t    *buffer,
-                                   unsigned int    context_length,
-                                   unsigned int    nesting_level_left,
-                                   unsigned int    property) const {
+  inline bool apply_subtables (hb_ot_layout_t *layout,
+                              hb_buffer_t    *buffer,
+                              unsigned int    context_length,
+                              unsigned int    nesting_level_left,
+                              unsigned int    property) const {
     unsigned int lookup_type = get_type ();
     unsigned int lookup_flag = get_flag ();
 
     for (unsigned int i = 0; i < get_subtable_count (); i++)
-      if (get_subtable (i).substitute (LOOKUP_ARGS, lookup_type))
+      if (get_subtable (i).apply (LOOKUP_ARGS, lookup_type))
        return true;
 
     return false;
   }
 
-  inline bool substitute_once (hb_ot_layout_t *layout, hb_buffer_t *buffer) const {
+  inline bool apply_once (hb_ot_layout_t *layout, hb_buffer_t *buffer) const {
 
     unsigned int lookup_flag = get_flag ();
 
@@ -718,12 +718,12 @@ struct SubstLookup : Lookup {
     if (!_hb_ot_layout_check_glyph_property (layout, IN_CURITEM (), lookup_flag, &property))
       return false;
 
-    return substitute_subtables (layout, buffer, NO_CONTEXT, MAX_NESTING_LEVEL, property);
+    return apply_subtables (layout, buffer, NO_CONTEXT, MAX_NESTING_LEVEL, property);
   }
 
-  bool substitute_string (hb_ot_layout_t *layout,
-                         hb_buffer_t    *buffer,
-                         hb_ot_layout_feature_mask_t mask) const {
+  bool apply_string (hb_ot_layout_t *layout,
+                    hb_buffer_t    *buffer,
+                    hb_ot_layout_feature_mask_t mask) const {
 
     bool ret = false;
 
@@ -738,7 +738,7 @@ struct SubstLookup : Lookup {
        while (buffer->in_pos < buffer->in_length) {
 
          if ((~IN_PROPERTIES (buffer->in_pos) & mask) &&
-             substitute_once (layout, buffer))
+             apply_once (layout, buffer))
            ret = true;
          else
            _hb_buffer_next_glyph (buffer);
@@ -754,7 +754,7 @@ struct SubstLookup : Lookup {
        do {
 
          if ((~IN_PROPERTIES (buffer->in_pos) & mask) &&
-             substitute_once (layout, buffer))
+             apply_once (layout, buffer))
            ret = true;
          else
            buffer->in_pos--;
@@ -786,7 +786,7 @@ struct GSUB : GSUBGPOS {
                                 hb_buffer_t    *buffer,
                                 unsigned int    lookup_index,
                                 hb_ot_layout_feature_mask_t  mask) const {
-    return get_lookup (lookup_index).substitute_string (layout, buffer, mask);
+    return get_lookup (lookup_index).apply_string (layout, buffer, mask);
   }
 
 };
@@ -795,13 +795,13 @@ ASSERT_SIZE (GSUB, 10);
 
 /* Out-of-class implementation for methods recursing */
 
-inline bool ExtensionSubstFormat1::substitute (LOOKUP_ARGS_DEF) const {
+inline bool ExtensionSubstFormat1::apply (LOOKUP_ARGS_DEF) const {
   unsigned int lookup_type = get_type ();
 
   if (HB_UNLIKELY (lookup_type ==  GSUB_Extension))
     return false;
 
-  return ((SubstLookupSubTable&)*(((char *) this) + get_offset ())).substitute (LOOKUP_ARGS, lookup_type);
+  return ((SubstLookupSubTable&)*(((char *) this) + get_offset ())).apply (LOOKUP_ARGS, lookup_type);
 }
 
 static inline bool substitute_lookup (LOOKUP_ARGS_DEF, unsigned int lookup_index) {
@@ -815,7 +815,7 @@ static inline bool substitute_lookup (LOOKUP_ARGS_DEF, unsigned int lookup_index
   if (HB_UNLIKELY (context_length < 1))
     return false;
 
-  return l.substitute_subtables (layout, buffer, context_length, nesting_level_left, property);
+  return l.apply_subtables (layout, buffer, context_length, nesting_level_left, property);
 }