<p dir="ltr">Am I reading it right that this patch was committed without a review by an approved reviewer (partly) because you weren't explicit enough?</p>
<div class="gmail_quote">07 янв. 2014 г. 3:51 пользователь "João Matos" <<a href="mailto:ripzonetriton@gmail.com">ripzonetriton@gmail.com</a>> написал:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr">Hi Timur,<div><br></div><div>No, I am not an approved reviewer, I only commented the changes to give some feedback about those changes, and maybe bring some attention to this forgotten patch.  There's an explicit option to accept the revision in Phab which I did not use.</div>


</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Jan 6, 2014 at 9:52 PM, Timur Iskhodzhanov <span dir="ltr"><<a href="mailto:timurrrr@google.com" target="_blank">timurrrr@google.com</a>></span> wrote:<br>


<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Joao,<br>
<br>
Are you an approved reviewer for this part of the codebase?<br>
<br>
2014/1/6 João Matos <<a href="mailto:ripzonetriton@gmail.com" target="_blank">ripzonetriton@gmail.com</a>>:<br>
<div>><br>
>   LGTM.<br>
><br>
>   I've tested the patch and was able to complete a C++11 LLVM build with CMake on OSX.<br>
><br>
> <a href="http://llvm-reviews.chandlerc.com/D2381" target="_blank">http://llvm-reviews.chandlerc.com/D2381</a><br>
</div>> _______________________________________________<br>
> llvm-commits mailing list<br>
> <a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br>João Matos
</div>
</blockquote></div>