From 38846f41d3760f9f70a7728b07c9829b776468a5 Mon Sep 17 00:00:00 2001
From: Garret Rieger <grieger@google.com>
Date: Fri, 29 Jul 2022 18:30:24 +0000
Subject: [PATCH] [repacker] more TODO cleanup.

---
 src/graph/gsubgpos-graph.hh | 3 ---
 src/graph/pairpos-graph.hh  | 4 ++--
 2 files changed, 2 insertions(+), 5 deletions(-)

diff --git a/src/graph/gsubgpos-graph.hh b/src/graph/gsubgpos-graph.hh
index f7dcc4bfd..afa1152c4 100644
--- a/src/graph/gsubgpos-graph.hh
+++ b/src/graph/gsubgpos-graph.hh
@@ -305,9 +305,6 @@ struct GSTAR : public OT::GSUBGPOS
     return len >= get_size ();
   }
 
-  // TODO(garretrieger): add find subtable's method, could be templated locate a specific
-  //                     subtable type, maybe take the lookup map as a starting point?
-
   void find_lookups (graph_t& graph,
                      hb_hashmap_t<unsigned, Lookup*>& lookups /* OUT */)
   {
diff --git a/src/graph/pairpos-graph.hh b/src/graph/pairpos-graph.hh
index 2b0183d64..7c905899f 100644
--- a/src/graph/pairpos-graph.hh
+++ b/src/graph/pairpos-graph.hh
@@ -230,13 +230,13 @@ struct PairPosFormat2 : public OT::Layout::GPOS_impl::PairPosFormat2_4<SmallType
 {
   bool sanitize (graph_t::vertex_t& vertex) const
   {
-    // TODO
+    // TODO(garretrieger): implement me!
     return true;
   }
 
   hb_vector_t<unsigned> split_subtables (gsubgpos_graph_context_t& c, unsigned this_index)
   {
-    // TODO
+    // TODO(garretrieger): implement me!
     return hb_vector_t<unsigned> ();
   }
 };