[PATCH 5/5] miinitext: introduce LoadExtensionList() to replace over LoadExtension()

Keith Packard keithp at keithp.com
Fri Feb 7 15:48:45 PST 2014


Emil Velikov <emil.l.velikov at gmail.com> 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 <keithp at keithp.com>

-- 
keith.packard at intel.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 810 bytes
Desc: not available
URL: <http://lists.x.org/archives/xorg-devel/attachments/20140207/e64f9b65/attachment.pgp>


More information about the xorg-devel mailing list