[PATCH 1/1] Look for ModuleData only in appropriate library
Daniel Stone
daniel at fooishbar.org
Wed Jun 27 10:01:55 PDT 2012
Hi,
On 27 June 2012 17:04, Michal Srb <msrb at suse.com> wrote:
> +void *
> +LoaderSymbolFromModule(const char *name, void *handle)
> +{
> + return dlsym(handle, name);
> +}
As a nitpick, I'd expect this to be LoaderSymbolFromModule(handle,
name) - and had wrote this even before I noticed that dlsym had the
arguments that way around too. It just seems like a more logical
order.
That aside:
Reviewed-by: Daniel Stone <daniel at fooishbar.org>
Cheers,
Daniel
More information about the xorg-devel
mailing list