<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Apr 28, 2014 at 7:52 PM, Nico Weber <span dir="ltr"><<a href="mailto:thakis@chromium.org" target="_blank">thakis@chromium.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">>> Let me know which one of the 3 patches (with 1 being the one I<br>
>> originally sent) you like best. If it's 3, I'll add a few more tests<br>
>> for that.<br>
><br>
><br>
> I strongly prefer patch 3 over 2. (With just patch 2, I worry that some libc<br>
> header would say "give me NULL and FOO", and just get NULL.)<br>
><br>
> Instead of repeating an 'ifdef _STDDEF_H_imp' test, could you define all the<br>
> __need_* macros in the _STDDEF_H_imp section? I'd find that slightly easier<br>
> to reason about.<br>
<br>
</div>Done. Also added a test, and fixed bugs found by the test.<br>
</blockquote></div><br></div><div class="gmail_extra">Please add a comment to the block where you #define __need_* saying we deliberately don't define __need_wint_t.</div><div class="gmail_extra"><br></div><div class="gmail_extra">
Other than that, the patch LGTM (though we'll need more changes here to make this work properly in the presence of modules).</div></div>