[PATCH] D45726: Format closing braces when reformatting the line containing theopening brace.

2018-04-23 Thread Manuel Klimek via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL330573: Format closing braces when reformatting the line containing the opening brace. (authored by klimek, committed by ). Herald added a subscriber: llvm-commits. Repository: rL LLVM

[PATCH] D45726: Format closing braces when reformatting the line containing theopening brace.

2018-04-20 Thread Manuel Klimek via Phabricator via cfe-commits
klimek updated this revision to Diff 143274. klimek marked 2 inline comments as done. klimek added a comment. Address comments. Repository: rC Clang https://reviews.llvm.org/D45726 Files: lib/Format/AffectedRangeManager.cpp lib/Format/AffectedRangeManager.h lib/Format/Format.cpp

[PATCH] D45726: Format closing braces when reformatting the line containing theopening brace.

2018-04-20 Thread Manuel Klimek via Phabricator via cfe-commits
klimek added a comment. In https://reviews.llvm.org/D45726#1071925, @krasimir wrote: > Another point: for the example in the summary about bailing-out early, is > there a test for this already? If not, we should add one. Yep, there already is one - I regressed that with my change at first ;)

[PATCH] D45726: Format closing braces when reformatting the line containing theopening brace.

2018-04-19 Thread Krasimir Georgiev via Phabricator via cfe-commits
krasimir added a comment. Another point: for the example in the summary about bailing-out early, is there a test for this already? If not, we should add one. Repository: rC Clang https://reviews.llvm.org/D45726 ___ cfe-commits mailing list