<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On 4 November 2015 at 12:37, Pino Toscano <span dir="ltr"><<a href="mailto:ptoscano@redhat.com" target="_blank">ptoscano@redhat.com</a>></span> wrote:</div><div class="gmail_quote">[...]<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">NACK, these OS-specific blocks are hard to maintain, and it makes no<br>
sense to enclose POSIX headers in them.<br>
<br>
Can you please try the attached patch? I see Rich already pushed your<br>
patch, but IMHO that should be reverted and this one should go...</blockquote><div><br></div><div style="font-size:12.8px">Happy to. However, these same header files have been removed in 3b17152ec399acb35b23331d2c3b4f424a68824a:</div><div style="font-size:12.8px"><br></div><div style="font-size:12.8px">commit 3b17152ec399acb35b23331d2c3b4f424a68824a</div><div style="font-size:12.8px">Author: Richard W.M. Jones <<a href="mailto:rjones@redhat.com" target="_blank">rjones@redhat.com</a>></div><div style="font-size:12.8px">Date:   Tue Sep 29 21:58:41 2015 +0100</div><div style="font-size:12.8px"><br></div><div style="font-size:12.8px">    lib: Remove unused header files.</div><div style="font-size:12.8px"><br></div><div style="font-size:12.8px">    I used ESR's deheader program to look for unused includes.  I then</div><div style="font-size:12.8px">    examined each instance by hand, and also test-compiled (on Linux).</div><div style="font-size:12.8px"><br></div><div><span style="font-size:12.8px">...which makes me suspect it will happen again at some point unless there is some indication that it's a non-Linux include.</span></div><div><br></div><div><br></div><div>-- </div><div>Margaret </div></div></div></div>