<div dir="ltr">Hi,<div class="gmail_extra"><br><div class="gmail_quote">On 8 April 2014 18:21, Kristian Høgsberg <span dir="ltr"><<a href="mailto:krh@bitplanet.net" target="_blank">krh@bitplanet.net</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">On Tue, Apr 8, 2014 at 10:13 AM, Daniel Stone <<a href="mailto:daniel@fooishbar.org">daniel@fooishbar.org</a>> wrote:<br>
> Doesn't this also need the syncshm.h include guarded?<br>
<br>
</div>No, the header is just the protoype for miSyncShmScreenInit().<br></blockquote><div><br></div><div>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. ;)</div>
<div><br></div><div>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:</div><div>Reviewed-by: Daniel Stone <<a href="mailto:daniel@fooishbar.org">daniel@fooishbar.org</a>></div>
<div><br></div><div>Cheers,</div><div>Daniel </div></div></div></div>