I don't know enough to give you an lgtm here, but I can say that I was having an issue with AlwaysBreakAfterDefinitionReturnType: true, BreakBeforeBraces: Allman, which is the style for LLDB, and this patch appears to fix my issue as well.
On Mon, Aug 11, 2014 at 8:50 AM, Curdeius Curdeius <[email protected]> wrote: > Hi, > > I have found a bug when using a style like: > AlwaysBreakAfterDefinitionReturnType: true > BreakBeforeBraces: Stroustrup > > I attach a straightforward patch that passes the tests. > 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). > > An example. > > Before: > template <class T> > T *f(T &c) // Problem here: no line break before f > { > return NULL; > } > > After: > template <class T> > T * > f(T &c) > { > return NULL; > } > > Regards, > Marek Kurdej > > _______________________________________________ > cfe-commits mailing list > [email protected] > http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits > >
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
