Re: Progress update on adjustment database

2023-07-30 Thread Werner LEMBERG
Hello Craig, again sorry for the late reply. > During this time, I realized that knowing the bounding box of the > tilde contour would help a lot. In fact, the logic for the other > vertical separation adjustments assumes that it can get the bounding > box by taking the minimum/maximum

CFF loading overhead in tracing

2023-07-30 Thread Behdad Esfahbod
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++ ) -

Re: Progress update on adjustment database

2023-07-30 Thread Craig White
> What exactly do you mean with 'find'? The algorithm? Just take the extrema of *all* points – for fonts this should be good enough, because it is standard to have points at the curve extrema, thus making the bounding box of the curve identical to the bounding box of the points (both on and off

Re: Progress update on adjustment database

2023-07-30 Thread Werner LEMBERG
> > Just take the extrema of *all* points – for fonts this should be > > good enough, because it is standard to have points at the curve > > extrema, thus making the bounding box of the curve identical to > > the bounding box of the points (both on and off points). > > Ok, thanks. This is

Re: CFF loading overhead in tracing

2023-07-30 Thread Behdad Esfahbod
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 wrote: > Hi, > > In benchmarking FreeType face loading, I noticed that the following two > loops take considerable time when loading CFF1 face: > > >

Re: ftbench update: make integrated

2023-07-30 Thread Werner LEMBERG
Here is my new bunch of remarks. In total, we are getting nearer to meaningful results :-) * The build process seems to be ok now, very good! * After doing `make baseline` (and `make benchmark`) I see as the last line ``` Processing 100%...\nBaseline created. ``` This `\n' looks