[Fontconfig] Excessive short-lived allocations during FcFontSet

Chris Wilson chris at chris-wilson.co.uk
Fri Apr 25 06:56:47 PDT 2008


On Wed, 2008-04-23 at 09:20 -0700, Keith Packard wrote:
> On Wed, 2008-04-23 at 09:38 +0100, Chris Wilson wrote:
> 
> > Please review, thanks.
> 
> Looks like a good bit of savings. A couple of comments:
> 
>  * I don't think we need to publish new API here.
> 
>  * It would be more efficient (and less confusing) if FcCharSetMerge
>    just did the union in-line -- it need only walk the new charset and
>    not look at each page of the existing one. Plus, that leaves all of
>    the other charset functions unchanged.

Thanks guys for your feedback. I've attached an updated patch.
-- 
Chris Wilson
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Reduce-number-of-allocations-during-FcSortWalk.patch
Type: application/mbox
Size: 0 bytes
Desc: not available
Url : http://lists.freedesktop.org/archives/fontconfig/attachments/20080425/2e70abd0/attachment.bin 


More information about the Fontconfig mailing list