<div dir="ltr">Thanks! Now I see the rule.</div><div class="gmail_extra"><br><br><div class="gmail_quote">2013/6/26 Richard Smith <span dir="ltr"><<a href="mailto:richard@metafoo.co.uk" target="_blank">richard@metafoo.co.uk</a>></span><br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Tue, Jun 25, 2013 at 7:31 AM, WenHan Gu (谷汶翰) <<a href="mailto:wenhan.gu@gmail.com">wenhan.gu@gmail.com</a>> wrote:<br>


> Thanks your review!<br>
><br>
> Hi Richard, I guess lib/Sema is owned by you. Could you help commit if ok?<br>
> Or who should I ask to help review and commit?<br>
<br>
</div>I ran the patch through clang-format (to fix an 80-column violation<br>
and change a "* " to a " *") and committed in r184875.<br>
<br>
You can ask anyone with Clang commit access to commit an approved<br>
patch for you; it doesn't need to be the code owner.<br>
<div class="HOEnZb"><div class="h5"><br>
> 2013/6/25 下午10:10 於 "Rafael Espíndola" <<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>> 寫道：<br>
><br>
>> LGTM<br>
>><br>
>> On 24 June 2013 21:35, WenHan Gu (谷汶翰) <<a href="mailto:wenhan.gu@gmail.com">wenhan.gu@gmail.com</a>> wrote:<br>
>> > Merge into existing testcase. Thanks!<br>
>> ><br>
>> > 2013/6/25 上午3:34 於 "Rafael Espíndola" <<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>> 寫道：<br>
>> ><br>
>> >> The test fails, since clang with -verify is looking for diagnostics to<br>
>> >> match. You need to add expected-no-diagnostics to the test or if<br>
>> >> possible merge the test with an existing file to reduce the number of<br>
>> >> distinct tests that get run during a check-clang invocation.<br>
>> >><br>
>> >> On 24 June 2013 14:34, WenHan Gu (谷汶翰) <<a href="mailto:wenhan.gu@gmail.com">wenhan.gu@gmail.com</a>> wrote:<br>
>> >> > Attached.<br>
>> >> > Sorry for inconvenience.<br>
>> >> ><br>
>> >> > 2013/6/24 下午10:20 於 "Rafael Espíndola" <<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>><br>
>> >> > 寫道：<br>
>> >> ><br>
>> >> >> Your second patch is a link to google docs and it is failing to<br>
>> >> >> download<br>
>> >> >> for me. Would you mind just attaching the patch?<br>
>> >> >><br>
>> >> >> Thanks,<br>
>> >> >> Rafael<br>
>> >> >><br>
>> >> ><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div dir="ltr">Best Regards,<div>WenHan Gu (Nowar)</div></div>
</div>