commit 02d4d4f3e67dcc37915bc386d506bb272455ff1e
author rsheeter <rsheeter@google.com> Wed May 08 14:43:18 2019 -0700
committer Rod Sheeter <rsheeter@google.com> Mon May 20 21:24:59 2019 -0700
tree 4b6c05baad3c4efb824e885ebe4db8eef6b4c128
parent ed972d5d73ba0592e1ba92426adf2a8f67acf9c9

[subset] Starting to sketch glyf as iter

src/hb-ot-glyf-table.hh

diff --git a/src/hb-ot-glyf-table.hh b/src/hb-ot-glyf-table.hh
index f7629ec..e30bdf7 100644
--- a/src/hb-ot-glyf-table.hh
+++ b/src/hb-ot-glyf-table.hh
@@ -81,24 +81,52 @@
     return_trace (true);
   }
 
-  bool subset (hb_subset_plan_t *plan) const
+  template <typename Iterator>
+  bool serialize(hb_serialize_context_t *c,
+		 Iterator it)
   {
-    hb_blob_t *glyf_prime = nullptr;
-    hb_blob_t *loca_prime = nullptr;
+    TRACE_SERIALIZE (this);
 
-    bool success = true;
-    bool use_short_loca = false;
-    if (hb_subset_glyf_and_loca (plan, &use_short_loca, &glyf_prime, &loca_prime)) {
-      success = success && plan->add_table (HB_OT_TAG_glyf, glyf_prime);
-      success = success && plan->add_table (HB_OT_TAG_loca, loca_prime);
-      success = success && _add_head_and_set_loca_version (plan, use_short_loca);
-    } else {
-      success = false;
-    }
-    hb_blob_destroy (loca_prime);
-    hb_blob_destroy (glyf_prime);
+    // TODO actually copy glyph
+    // TODO worry about instructions
+    // TODO capture dest locations for loca
 
-    return success;
+    return_trace (true);
+  }
+
+  bool subset (hb_subset_context_t *c) const
+  {
+    TRACE_SUBSET (this);
+
+    glyf *glyf_prime = c->serializer->start_embed <glyf> ();
+    if (unlikely (!glyf_prime)) return_trace (false);
+
+    OT::glyf::accelerator_t glyf;
+    glyf.init (c->plan->source);
+
+    // stream new-gids => pair of start/end offsets
+    // can now copy glyph from <prev>=>end
+    // TODO(instructions)
+    auto it =
+    + hb_iota (c->plan->num_output_glyphs ())
+    | hb_map ([&] (hb_codepoint_t new_gid) {
+      unsigned int start_offset = 0, end_offset = 0;
+      // simple case: empty glyph
+      hb_codepoint_t old_gid;
+      if (!c->plan->old_gid_for_new_gid (new_gid, &old_gid)) return hb_pair (start_offset, end_offset);
+
+      if (unlikely (!(glyf.get_offsets (old_gid, &start_offset, &end_offset) &&
+	glyf.remove_padding (start_offset, &end_offset))))
+      {
+	// TODO signal irreversible error
+	return hb_pair (start_offset, end_offset);
+      }
+      return hb_pair (start_offset, end_offset);
+    });
+
+    glyf_prime->serialize (c->serializer, it);
+
+    return_trace (true);
   }
 
   static bool

src/hb-subset-plan.hh

diff --git a/src/hb-subset-plan.hh b/src/hb-subset-plan.hh
index abbab5e..bdc7101 100644
--- a/src/hb-subset-plan.hh
+++ b/src/hb-subset-plan.hh
@@ -68,6 +68,8 @@
 
   /*
    * The set of input glyph ids which will be retained in the subset.
+   * Does NOT include ids kept due to retain_gids. You probably want to use
+   * glyph_map/reverse_glyph_map.
    */
   inline const hb_set_t *
   glyphset () const

src/hb-subset.cc

diff --git a/src/hb-subset.cc b/src/hb-subset.cc
index f4fc771..ab9b4d8 100644
--- a/src/hb-subset.cc
+++ b/src/hb-subset.cc
@@ -156,7 +156,7 @@
   bool result = true;
   switch (tag) {
     case HB_OT_TAG_glyf:
-      result = _subset<const OT::glyf> (plan);
+      result = _subset2<const OT::glyf> (plan);
       break;
     case HB_OT_TAG_hdmx:
       result = _subset2<const OT::hdmx> (plan);