[PATCH] D13811: [clang-format] AllowShortFunctionsOnASingleLine: true/Empty didn't work with BreakBeforeBraces: Linux/Allman.

2017-09-20 Thread Marek Kurdej via Phabricator via cfe-commits
curdeius abandoned this revision. curdeius added a comment. This was fixed by https://reviews.llvm.org/rL312904 and other commits. https://reviews.llvm.org/D13811 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

Re: [PATCH] D13811: [clang-format] AllowShortFunctionsOnASingleLine: true/Empty didn't work with BreakBeforeBraces: Linux/Allman.

2015-11-09 Thread Marek Kurdej via cfe-commits
curdeius added a comment. Ping? http://reviews.llvm.org/D13811 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D13811: [clang-format] AllowShortFunctionsOnASingleLine: true/Empty didn't work with BreakBeforeBraces: Linux/Allman.

2015-10-20 Thread Marek Kurdej via cfe-commits
curdeius updated this revision to Diff 37832. curdeius added a comment. - AllowShortFunctionsOnASingleLine: true didn't work with BreakBeforeBraces: Linux/Allman. - Add test checking that non-empty functions in styles with `BraceWrapping.AfterFunction = true` don't get merged into one line. Fix