Humm. I cannot reproduce this anymore. Will report if that changes. behdad http://behdad.org/
On Sun, Jul 30, 2023 at 12:41 PM Behdad Esfahbod <beh...@behdad.org> wrote: > 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/ >
