<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On 10 March 2014 13:59, Justin Bogner <span dir="ltr"><<a href="mailto:mail@justinbogner.com" target="_blank">mail@justinbogner.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">Viktor Kutuzov <<a href="mailto:vkutuzov@accesssoftek.com">vkutuzov@accesssoftek.com</a>> writes:<br>


>   Updated so that it does not change the set of inclusions on Windows,<br>
>   does not duplicate headers and pays more attention to scalability.<br>
><br>
>   Inspiried by comments in llvm-commits by Justin Bogner:<br>
><br>
>   > Are sys/stat.h and stdint.h not used for some reason in the<br>
>   > _MSC_VER case? This looks like a behaviour change that probably<br>
>   > breaks windows.<br>
>   > Duplicating the include of inttypes in _MSC_VER and in the else<br>
>   > seems fairly awkward here.<br>
>   > Overall, this approach really doesn't seem scalable at all, and it<br>
>   > will be very difficult for people adding things in this area to<br>
>   > get it right without submitting to buildbots.<br>
><br>
>   Thanks, Justin.<br>
><br>
>   If this version looks OK, I would appreciate if one the committers<br>
>   land it as I don't have the commit access. Thanks.<br>
<br>
</div>I'd missed this update, so I apologize for the wait, but I notice that<br>
you've committed this in the meantime without waiting for an OK.<br>
<br>
Please don't do that in the future.<br>
<br>
It's important not to commit code until all reviewers that have raised<br>
concerns have acknowledged that their concerns have been addressed.<br></blockquote><div><br></div><div>FYI, I had intended my email on Feb 24th to be an LGTM (it shows up as "approved" in the code review tool though the email doesn't say that). I haven't been following the progression of the thread after that point.</div>

</div></div></div>