commit a97ce570ab693190350886e4e80942851c4d5727
author Behdad Esfahbod <behdad@behdad.org> Tue Aug 04 02:10:48 2009 -0400
committer Behdad Esfahbod <behdad@behdad.org> Mon Nov 02 14:40:28 2009 -0500
tree 9defbeaf154b4ddfc1aa138e89e611d79d3300fd
parent cd3827ee567612c5500206b62840702fc956e0f5

[HB] Optimize sanitize()

src/hb-open-types-private.hh

diff --git a/src/hb-open-types-private.hh b/src/hb-open-types-private.hh
index 3d57122..b3d7541 100644
--- a/src/hb-open-types-private.hh
+++ b/src/hb-open-types-private.hh
@@ -376,11 +376,12 @@
 
   inline bool sanitize (SANITIZE_ARG_DEF) {
     if (!(SANITIZE (len) && SANITIZE_GET_SIZE())) return false;
-    /* XXX For non-recursive types, this is too much overhead */
+    /* Note; for non-recursive types, this is not much needed
     unsigned int count = len;
     for (unsigned int i = 0; i < count; i++)
       if (!SANITIZE (array[i]))
         return false;
+    */
   }
 
   USHORT len;
@@ -403,9 +404,11 @@
   inline bool sanitize (SANITIZE_ARG_DEF) {
     if (!(SANITIZE_SELF () && SANITIZE_GET_SIZE())) return false;
     unsigned int count = len ? len - 1 : 0;
+    /* Note; for non-recursive types, this is not much needed
     for (unsigned int i = 0; i < count; i++)
       if (!SANITIZE (array[i]))
         return false;
+    */
   }
 
   USHORT len;
@@ -427,9 +430,11 @@
   inline bool sanitize (SANITIZE_ARG_DEF) {
     if (!(SANITIZE_SELF () && SANITIZE_GET_SIZE())) return false;
     unsigned int count = len;
+    /* Note; for non-recursive types, this is not much needed
     for (unsigned int i = 0; i < count; i++)
       if (!SANITIZE (array[i]))
         return false;
+    */
   }
 
   ULONG len;