From 417963dd9f93a316e672c77493e763978bfdbc28 Mon Sep 17 00:00:00 2001
From: Michiharu Ariza <ariza@adobe.com>
Date: Thu, 8 Nov 2018 09:01:20 -0800
Subject: [PATCH] got rid of Remap::fullset() as incorrect, redundant,
 inefficient

---
 src/hb-ot-cff-common.hh | 22 ----------------------
 1 file changed, 22 deletions(-)

diff --git a/src/hb-ot-cff-common.hh b/src/hb-ot-cff-common.hh
index 4607efb24..5f98469bf 100644
--- a/src/hb-ot-cff-common.hh
+++ b/src/hb-ot-cff-common.hh
@@ -433,34 +433,12 @@ struct Remap : hb_vector_t<hb_codepoint_t>
     return true;
   }
 
-  inline bool fullset (void) const
-  {
-    for (unsigned int i = 0; i < len; i++)
-      if (SUPER::operator[] (i) == CFF_UNDEF_CODE)
-        return false;
-    return true;
-  }
-
   inline bool excludes (hb_codepoint_t id) const
   { return (id < len) && ((*this)[id] == CFF_UNDEF_CODE); }
 
   inline bool includes (hb_codepoint_t id) const
   { return !excludes (id); }
 
-  inline hb_codepoint_t operator[] (hb_codepoint_t i) const
-  {
-    if (fullset ())
-      return i;
-    else
-      return SUPER::operator[] (i);
-  }
-
-  inline hb_codepoint_t &operator[] (hb_codepoint_t i)
-  {
-    assert (i < len);
-    return SUPER::operator[] (i);
-  }
-
   inline unsigned int add (unsigned int i)
   {
     if ((*this)[i] == CFF_UNDEF_CODE)