<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Nov 21, 2013 at 4:04 PM, Alexander Kornienko <span dir="ltr"><<a href="mailto:alexfh@google.com" target="_blank">alexfh@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"><div class="gmail_quote"><div class="im">On Thu, Nov 21, 2013 at 3:56 PM, Daniel Jasper <span dir="ltr"><<a href="mailto:djasper@google.com" target="_blank">djasper@google.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><p dir="ltr">I don't feel strongly about this. However, putting then decision into mustBreak would just work. You don't need any special casing for the single-line case as that is handled by a different code path..</p>

</blockquote></div><div>You mean putting all the tryMerge.* methods to mustBreak?</div></div></div></div></blockquote><div><br></div><div>I.e. the logic implemented in tryMerge methods.</div><div> </div></div>
</div></div>