<div dir="ltr"><div class="gmail_default" style>Changed in r172431.</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Jan 14, 2013 at 5:26 PM, Tobias Grosser <span dir="ltr"><<a href="mailto:tobias@grosser.es" target="_blank">tobias@grosser.es</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On 01/14/2013 04:47 PM, Douglas Gregor wrote:<br>
</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">
<br>
On Jan 14, 2013, at 7:43 AM, Daniel Jasper <<a href="mailto:djasper@google.com" target="_blank">djasper@google.com</a><br></div><div class="im">
<mailto:<a href="mailto:djasper@google.com" target="_blank">djasper@google.com</a>>> wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Sure. I'll add this as an option and add a default style for Chromium<br>
(we have other stuff we want to configure according to the Chromium<br>
style guide).<br>
</blockquote>
<br>
Thank you!<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Douglas: Should I set this to break or no break for the default LLVM<br>
Style?<br>
</blockquote>
<br>
Unfortunately, the style guide is itself inconsistent. Personally, I'd<br>
prefer to break there.<br>
</div></blockquote>
<br>
I agree with Doug here.<br>
<br>
>From reading LLVM code it seems that ifs without breaks exists, but to me they always look like something special.<br>
<br>
All the best,<br>
Tobi<br>
<br>
</blockquote></div><br></div>