CFF loading overhead in tracing

Hi,

In benchmarking FreeType face loading, I noticed that the following two
loops take considerable time when loading CFF1 face:


--- a/src/cff/cffobjs.c
+++ b/src/cff/cffobjs.c
@@ -695,8 +695,8 @@


           FT_TRACE4(( "  %5d ", idx + 390 ));
-          for ( l = 0; l < s1len; l++ )
-            FT_TRACE4(( "%c", s1[l] ));
+          //for ( l = 0; l < s1len; l++ )
+            //FT_TRACE4(( "%c", s1[l] ));
           FT_TRACE4(( "\n" ));
         }

@@ -710,8 +710,8 @@


           FT_TRACE4(( "  %5d ", cff->num_strings + 390 ));
-          for ( l = 0; l < s1len; l++ )
-            FT_TRACE4(( "%c", s1[l] ));
+          //for ( l = 0; l < s1len; l++ )
+            //FT_TRACE4(( "%c", s1[l] ));
           FT_TRACE4(( "\n" ));
         }
       }


I don't have tracing enabled, but the compiler doesn't seem to have wiped
away the loop. I'm not sure why. At any rate, I wonder if these can be
guarded by FT_DEBUG_TRACE or some other way to remove the overhead?


behdad
http://behdad.org/