On Wed, Nov 20, 2013 at 6:12 PM, Daniel Jasper <[email protected]> wrote:
>
>> @@ -681,6 +681,7 @@ void UnwrappedLineParser::parseStructura
>> Style.BreakBeforeBraces == FormatStyle::BS_Stroustrup ||
>> Style.BreakBeforeBraces == FormatStyle::BS_Allman)
>> addUnwrappedLine();
>>
>
> Does it still make sense to report the "{" as its own unwrapped line?
> Seems a bit convoluted to first report multiple lines and then merge them
> afterwards. I think this would make the merging code simpler.
>
It also seemed strange to me. Should we instead handle BreakBeforeBraces in
TokenAnnotator? This will require adding TokenType values for braces
starting namespaces, classes/structs and, probably, enums. I can play with
this a bit, it you think it makes sense.
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
