commit b648ceb72f281e739ec8bd73cbc243624854238d
author Behdad Esfahbod <behdad@behdad.org> Sat Jan 21 15:58:59 2023 -0700
committer Behdad Esfahbod <behdad@behdad.org> Sat Jan 21 15:58:59 2023 -0700
tree 3c7468b51e1fc3006b349ad06728c06179c13581
parent 891623243c167217fcbd9480b111b110cc004c9c

[PairSet] Optimize last commit

src/OT/Layout/GPOS/PairSet.hh

diff --git a/src/OT/Layout/GPOS/PairSet.hh b/src/OT/Layout/GPOS/PairSet.hh
index 49e97b0..147b8e0 100644
--- a/src/OT/Layout/GPOS/PairSet.hh
+++ b/src/OT/Layout/GPOS/PairSet.hh
@@ -24,11 +24,15 @@
   public:
   DEFINE_SIZE_MIN (2);
 
+  static unsigned get_size (unsigned len1, unsigned len2)
+  {
+    return Types::HBGlyphID::static_size + Value::static_size * (len1 + len2);
+  }
   static unsigned get_size (const ValueFormat valueFormats[2])
   {
     unsigned len1 = valueFormats[0].get_len ();
     unsigned len2 = valueFormats[1].get_len ();
-    return Types::HBGlyphID::static_size + Value::static_size * (len1 + len2);
+    return get_size (len1, len2);
   }
 
   struct sanitize_closure_t
@@ -101,7 +105,7 @@
     hb_buffer_t *buffer = c->buffer;
     unsigned int len1 = valueFormats[0].get_len ();
     unsigned int len2 = valueFormats[1].get_len ();
-    unsigned record_size = get_size (valueFormats);
+    unsigned record_size = get_size (len1, len2);
 
     const PairValueRecord *record = hb_bsearch (buffer->info[pos].codepoint,
                                                 &firstPairValueRecord,
@@ -167,7 +171,8 @@
     const hb_map_t &glyph_map = *c->plan->glyph_map;
 
     unsigned len1 = valueFormats[0].get_len ();
-    unsigned record_size = get_size (valueFormats);
+    unsigned len2 = valueFormats[1].get_len ();
+    unsigned record_size = get_size (len1, len2);
 
     typename PairValueRecord::context_t context =
     {