Hi,

On 8 April 2014 18:21, Kristian Høgsberg <[email protected]> wrote:

> On Tue, Apr 8, 2014 at 10:13 AM, Daniel Stone <[email protected]>
> wrote:
> > Doesn't this also need the syncshm.h include guarded?
>
> No, the header is just the protoype for miSyncShmScreenInit().
>

Oh sure, was just that the last time I reviewed xwayland patches, it was
built externally where we couldn't rely on that being installed. ;)

Would still be nice to make the header conditional so we'll get screaming
compiler death if we ever leak that use back out of an ifdef, but eh,
either way:
Reviewed-by: Daniel Stone <[email protected]>

Cheers,
Daniel
_______________________________________________
[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