<div dir="ltr">This was decided to be the best way to go forward with Google's C++ style. I am reasonably certain that the style guide does not contain anything that says otherwise. The section you quote:<div>a) applies to boolean expressions, which a conditional expression is not.</div>
<div>b) explicitly says that you can also wrap them to the new line.</div><div><br></div><div>If you find anything else in the style guide, please let me know, I am happy to get that changed.</div><div><br></div><div>As for "This is not done in Chromium":</div>
<div><a href="https://code.google.com/p/chromium/codesearch#search/&q=%5E%5C%20*%5C?&all=1&sq=package:chromium&type=cs" class="cremed">https://code.google.com/p/chromium/codesearch#search/&q=%5E%5C%20*%5C?&all=1&sq=package:chromium&type=cs</a><br>
</div><div>and</div><div><a href="https://code.google.com/p/chromium/codesearch#search/&q=(?m:%5C?.*%5Cn%5Cs*%5C:)%20pcre:yes&all=1&sq=package:chromium&type=cs" class="cremed">https://code.google.com/p/chromium/codesearch#search/&q=(?m:%5C?.*%5Cn%5Cs*%5C:)%20pcre:yes&all=1&sq=package:chromium&type=cs</a><br>
</div><div><br></div><div>I am aware that the other way of formatting is ~2:1 more common in Chromium right now, but there already is quite a bit of diversity.</div><div><br></div><div>The basic reasoning for going with the slightly less popular style is this very basic example:</div>
<div><br></div><div><font face="courier new, monospace">aaaaa ? bbbbb :</font></div><div><font face="courier new, monospace">        ccccc;</font></div><div><font face="courier new, monospace"><br></font></div><div><font face="courier new, monospace">aaaaa ? bbbbb</font></div>
<div><font face="courier new, monospace">      : ccccc;</font></div><div><br></div><div>Here, the second formatting looks more structured and is easier to grasp at first sight. If the operands get more complex, this gets even more obvious. And for consistency with that formatting, ? and : should both be wrapped to the new line where necessary.</div>
<div><br></div><div>Cheers,<br>Daniel</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Sat, Aug 10, 2013 at 9:06 PM, Thiago Farina <span dir="ltr"><<a href="mailto:tfransosi@gmail.com" target="_blank" class="cremed">tfransosi@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<br>
Looks like clang-format for Chromium style is incorrectly formatting<br>
ternary operators (?:)<br>
<br>
By Google C++ Style guide, which Chromium code follows:<br>
<a href="http://google-styleguide.googlecode.com/svn/trunk/cppguide.xml?showone=Boolean_Expressions#Boolean_Expressions" target="_blank" class="cremed">http://google-styleguide.googlecode.com/svn/trunk/cppguide.xml?showone=Boolean_Expressions#Boolean_Expressions</a><br>

<br>
The operators should be in the end of the expressions.<br>
<br>
But clang-format did this:<br>
<a href="https://codereview.chromium.org/21696003/diff/35001/chrome/browser/ui/views/location_bar/location_bar_view.cc#newcode1107" target="_blank" class="cremed">https://codereview.chromium.org/21696003/diff/35001/chrome/browser/ui/views/location_bar/location_bar_view.cc#newcode1107</a><br>

<br>
According to the style guide and Peter it should have been formatted like this:<br>
<br>
return (ui::GetDisplayLayout() == ui::LAYOUT_TOUCH) ?<br>
    GetItemPadding() / 2 : 0;<br>
<br>
--<br>
Thiago<br>
</blockquote></div><br></div></div>