commit 209491fc37c46281e063c3e6707d686d5f2b2ba4
author Qunxin Liu <qxliu@google.com> Wed May 29 14:03:17 2019 -0700
committer Garret Rieger <grieger@google.com> Tue Jun 04 12:08:05 2019 -0700
tree 19c7820c26fdb19c9c70d778c448fe724bda7566
parent 89a7a880a1d5cd5e585d43fdde6d44c6dba559cf

[subset] Hmtx/vmtx tables to use subset2 and new iterator frameworks

src/hb-subset.cc

diff --git a/src/hb-subset.cc b/src/hb-subset.cc
index 7943695..ddd88f7 100644
--- a/src/hb-subset.cc
+++ b/src/hb-subset.cc
@@ -67,7 +67,7 @@
 static bool
 _subset2 (hb_subset_plan_t *plan)
 {
-  bool result = true;
+  bool result = false;
   hb_blob_t *source_blob = hb_sanitize_context_t ().reference_table<TableType> (plan->source);
   const TableType *table = source_blob->as<TableType> ();
 
@@ -172,13 +172,13 @@
       DEBUG_MSG(SUBSET, nullptr, "skip hhea handled by hmtx");
       return true;
     case HB_OT_TAG_hmtx:
-      result = _subset<const OT::hmtx> (plan);
+      result = _subset2<const OT::hmtx> (plan);
       break;
     case HB_OT_TAG_vhea:
       DEBUG_MSG(SUBSET, nullptr, "skip vhea handled by vmtx");
       return true;
     case HB_OT_TAG_vmtx:
-      result = _subset<const OT::vmtx> (plan);
+      result = _subset2<const OT::vmtx> (plan);
       break;
     case HB_OT_TAG_maxp:
       result = _subset<const OT::maxp> (plan);