<div dir="ltr"><div class="gmail_default" style>I see. I first thought the same. But switching forward and back between the two, I actually think that the new one is at least as readable. Or at least I can't say that one is strictly better then the other. Maybe we'll find other examples where the decision is more obvious..</div>
</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Jan 7, 2013 at 2:33 PM, Chandler Carruth <span dir="ltr"><<a href="mailto:chandlerc@google.com" target="_blank">chandlerc@google.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"><div class="gmail_extra">FYI</div><div class="im"><div class="gmail_extra"><br></div><div class="gmail_extra">
On Mon, Jan 7, 2013 at 5:26 AM, Daniel Jasper <span dir="ltr"><<a href="mailto:djasper@google.com" target="_blank">djasper@google.com</a>></span> wrote:<br>
<div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div>-      FormatTok.HasUnescapedNewline =<br>
-          Text.count("\\\n") != FormatTok.NewlinesBefore;<br>
+      FormatTok.HasUnescapedNewline = Text.count("\\\n") !=<br>
+                                      FormatTok.NewlinesBefore;</div></blockquote></div><br></div></div><div class="gmail_extra">This doesn't seem as good as the rest of the changes. Here I agree with your strategy of breaking the outer most binary operator. That's why I was trying to differentiate between function arguments being broken after an '=' and breaking a binary operator...</div>

</div>
</blockquote></div><br></div>