[HarfBuzz] harfbuzz: Branch 'master' - 2 commits

Behdad Esfahbod behdad at kemper.freedesktop.org
Sun Oct 15 21:02:39 UTC 2017


 src/hb-set-private.hh |   10 ++--------
 1 file changed, 2 insertions(+), 8 deletions(-)

New commits:
commit 0cf7d957b7ffb17c4cec75b631c9829327ab5d52
Author: Behdad Esfahbod <behdad at behdad.org>
Date:   Sun Oct 15 17:01:31 2017 -0400

    [set] Remove TODO items not worth pursuing

diff --git a/src/hb-set-private.hh b/src/hb-set-private.hh
index 9cd15d5a..a7ca4f4a 100644
--- a/src/hb-set-private.hh
+++ b/src/hb-set-private.hh
@@ -224,7 +224,6 @@ struct hb_set_t
   inline void del_range (hb_codepoint_t a, hb_codepoint_t b)
   {
     if (unlikely (in_error)) return;
-    /* TODO Speedup */
     for (unsigned int i = a; i < b + 1; i++)
       del (i);
   }
@@ -396,7 +395,6 @@ struct hb_set_t
   {
     hb_codepoint_t i;
 
-    /* TODO speedup. */
     i = *last;
     if (!next (&i))
     {
commit 10d43658e7655b36cb86e674dde8af4036b7cc52
Author: Behdad Esfahbod <behdad at behdad.org>
Date:   Sun Oct 15 16:56:05 2017 -0400

    [set] Speed up intersects()

diff --git a/src/hb-set-private.hh b/src/hb-set-private.hh
index 7bb57d9f..9cd15d5a 100644
--- a/src/hb-set-private.hh
+++ b/src/hb-set-private.hh
@@ -238,12 +238,8 @@ struct hb_set_t
   inline bool intersects (hb_codepoint_t first,
 			  hb_codepoint_t last) const
   {
-    /* TODO Speedup */
-    unsigned int end = last + 1;
-    for (hb_codepoint_t i = first; i < end; i++)
-      if (has (i))
-        return true;
-    return false;
+    hb_codepoint_t c = first - 1;
+    return next (&c) && c <= last;
   }
   inline void set (const hb_set_t *other)
   {


More information about the HarfBuzz mailing list