Emil Velikov <[email protected]> writes:

> Looping around LoadExtension() meant that ExtensionModuleList was reallocated
> on every extension. Using LoadExtensionList() we pass an array thus the
> function can do the reallocation in one go, and then loop and setup the
> ExtensionModuleList.

Nice clean up; the realloc cost isn't that important, but getting rid of
identical loops in a bunch of places is definitely an improvement.

Reviewed-by: Keith Packard <[email protected]>

-- 
[email protected]

Attachment: pgpiDd3cCNFGt.pgp
Description: PGP signature

_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to