[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