<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Oct 22, 2013 at 10:33 AM, Serge Pavlov <span dir="ltr"><<a href="mailto:sepavloff@gmail.com" target="_blank" class="cremed">sepavloff@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">This problem has been fixed in r193170.<div><br><div>I regret about troubles. GCC rejects very similar syntax and without a proper test it was very simple to break software that uses this strange feature.</div>


<div>I believe the fix is useful and change in r193170 will be enough to pacify QT world.</div></div></div></blockquote></div><br>You never answered who reviewed this. I plan to back all of this out until you get pre-commit review.</div>
<div class="gmail_extra"><br></div><div class="gmail_extra">You also didn't put the QT test case back. You should. We actually have customers relying on Clang supporting QT which is why we had test cases for it. Please don't remove test cases that you don't understand.</div>
<div class="gmail_extra"><br></div><div class="gmail_extra">Please start a review thread for your fix to this PR rather than committing directly.</div></div>