<div dir="ltr">On Mon, Aug 12, 2013 at 1:57 PM, Daniel Jasper <span dir="ltr"><<a href="mailto:djasper@google.com" target="_blank">djasper@google.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div class="im"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div>Finally, regardless of all other arguments, and even ignoring the fact that the above codesearch links included code they shouldn't: there's a way of formatting these in Chromium code that is clearly more common even by a pessimistic search, and is clearly compliant with the style guide.  Therefore, the auto-formatter for Chromium code should use this pattern.  It doesn't matter if your way is legal or not, the most important rule in the whole style guide is "be consistent", which this does not do as well as it could.<br>
</div></div></div></div></blockquote></div><div class="im"><div><br></div></div><div>I understand your argument and it is perfectly valid. However, also consider other sorts of consistency: E.g. there are a lot of users that have to develop in both Chromium- and Google-style code. For them, any (unnecessary) inconsistency is harmful.</div>
</div></div></div></blockquote><div><br></div><div>It seems clearly far more important that Chromium code be internally consistent than that the formatter begin preferring a new, less-used style over an existing, common, explicitly-valid style regardless of which way is common in Google internal code.  This is true within other Google projects as well.</div>
<div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">
<div>I personally don't have any strong feelings about this (I for one would be happy with disallowing all multi-line conditional expressions). The decision to go this way is mostly that complex conditional expressions need as much structure as they can get. I know that we have other Chromium engineers that are happy enough with this. Is there a decision making process for Chromium style?</div>
</div></div></div></blockquote><div><br></div><div>I don't see what "structure" you're buying with this style; it seems like this:</div><div><br></div><div><font face="courier new, monospace">aaaaa ?</font></div>
<div><font face="courier new, monospace">    bbbbb :</font></div><div><font face="courier new, monospace">    ccccc;</font></div><div><font face="courier new, monospace"><br></font></div><div><font face="arial, helvetica, sans-serif">...is just as "structured" as what you're asking for, but aligns with the common Chromium idiom.</font></div>
<div><br></div><div>If you are not willing to accept my argument on its face and simply make this change, the next step is to escalate to chromium-dev.</div><div><br></div><div>But in the case where you "don't have any strong feelings" and I do, and I've been working on the codebase for 7 years now, and you accept my argument that the style I'm asking for is valid, I don't see what you gain by doing so.</div>
<div><br>PK</div></div></div></div>