<div dir="ltr">I don't know enough to give you an lgtm here, but I can say that I was having an issue with <span style="font-family:arial,sans-serif;font-size:13px">AlwaysBreakAfterDefinitionRetu</span><span style="font-family:arial,sans-serif;font-size:13px">rnType: true, </span><span style="font-family:arial,sans-serif;font-size:13px">BreakBeforeBraces: Allman, which is the style for LLDB, and this patch appears to fix my issue as well.</span></div>
<div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Aug 11, 2014 at 8:50 AM, Curdeius Curdeius <span dir="ltr"><<a href="mailto:curdeius@gmail.com" target="_blank">curdeius@gmail.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>Hi,</div><div><br></div><div>I have found a bug when using a style like:</div><div>  AlwaysBreakAfterDefinitionReturnType: true</div>
<div>  BreakBeforeBraces: Stroustrup</div><div><br></div><div>I attach a straightforward patch that passes the tests.</div>

<div>However, I think that the issue may be actually somewhere deeper, e.g. in `Line.Last` pointing incorrectly to a different token than l_brace (cf. the patch).</div><div><br></div><div>An example.</div><div><br></div>

<div>
<div>Before:</div><div>  template <class T></div><div>  T *f(T &c)  // Problem here: no line break before f</div><div>  {</div><div>    return NULL;</div><div>  }</div><div><br></div></div><div><div>After:</div>


<div>  template <class T></div><div>  T *</div><div>  f(T &c)</div><div>  {</div><div>    return NULL;</div><div>  }</div></div><div><br></div><div>Regards,<br>Marek Kurdej</div>
</div>
<br>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div><br></div>