[PATCH 3/4] Include config.h before any other headers.

Emil Velikov emil.l.velikov at gmail.com
Sat Feb 7 10:16:49 PST 2015


On Saturday, 7 February 2015, Alan Coopersmith <alan.coopersmith at oracle.com>
wrote:

> Reviewed-by: Alan Coopersmith <alan.coopersmith at oracle.com>
>
> Though I wonder if now you can drop the explicit #define _GNU_SOURCE in
> linux_sysfs.c now that you moved the config.h up to the same spot, since
> the AC_USE_SYSTEM_EXTENSIONS in configure.ac should #define _GNU_SOURCE in
> config.h.
>
> Thanks for the review Alan.

If you don't mind I'll do the removal as a follow up patch.  Will give
these another 2 days and I'll resend the series with all the comments
addressed + correct prefix.

-Emil
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.x.org/archives/xorg-devel/attachments/20150207/27b027de/attachment.html>


More information about the xorg-devel mailing list