<div dir="rtl"><div dir="ltr">Hi Eugene,</div><div dir="ltr"><br></div><div dir="ltr">These patches LGTM. Should I commit them for you?</div><div dir="ltr"><br></div><div dir="ltr">Yaron</div><div dir="ltr"><br></div></div><div class="gmail_extra"><br><div class="gmail_quote"><div dir="ltr">2015-06-26 23:33 GMT+03:00 Eugene Kosov <span dir="ltr"><<a href="mailto:claprix@yandex.ru" target="_blank">claprix@yandex.ru</a>></span>:</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi.<br>
<br>
Thanks for your feedback. I fixed the regression. Here is the new patch.<br>
<br>
26.06.2015, 00:51, "Yaron Keren" <<a href="mailto:yaron.keren@gmail.com">yaron.keren@gmail.com</a>>:<br>
<div><div class="h5">> Hi Eugene,<br>
><br>
> This patch does not pass clang extra tools regression tests locally,<br>
><br>
> FAIL: Clang Tools :: modularize/NoProblems.modularize (7141 of 19860)<br>
> Testing: 0 .. 10.. 20.. 30<br>
> FAIL: Clang Tools :: modularize/NoProblemsDependencies.modularize (7144 of 19860)<br>
> Testing: 0 .. 10.. 20.. 30<br>
> FAIL: Clang Tools :: modularize/NoProblemsCoverage.modularize (7145 of 19860)<br>
> (...)<br>
><br>
> Please check it out.<br>
><br>
> Yaron<br>
><br>
> 2015-06-22 15:25 GMT+03:00 Eugene Kosov <<a href="mailto:claprix@yandex.ru">claprix@yandex.ru</a>>:<br>
>> Hi.<br>
>><br>
>> Here are two patches for clang-tools-extra and for clang which simplifies code a little bit and also removes possibility of implicit std::string construction.<br>
>><br>
>> --<br>
>> Eugene<br>
>><br>
>> _______________________________________________<br>
>> cfe-commits mailing list<br>
>> <a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
>> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br>
</div></div>--<br>
Eugene<br>
</blockquote></div><br></div>