[PATCH] D29943: [clang-format] Align block comment decorations

2017-02-20 Thread Krasimir Georgiev via Phabricator via cfe-commits
krasimir added a comment. In https://reviews.llvm.org/D29943#678618, @sylvestre.ledru wrote: > Maybe this could be added to the release notes? Sounds good! Could you please point me to the release docs? I don't seem to find a clang-format--specific section online. Repository: rL LLVM http

[PATCH] D29943: [clang-format] Align block comment decorations

2017-03-14 Thread Sylvestre Ledru via Phabricator via cfe-commits
sylvestre.ledru added a comment. I did it for you Added to the release notes in r297721 Repository: rL LLVM https://reviews.llvm.org/D29943 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe

[PATCH] D29943: [clang-format] Align block comment decorations

2017-02-16 Thread Manuel Klimek via Phabricator via cfe-commits
klimek added inline comments. Comment at: lib/Format/BreakableToken.cpp:397 +// +// Don't try aligning if there are less lines, since some patterns like +// fewer lines Comment at: lib/Format/BreakableToken.cpp:402 +

[PATCH] D29943: [clang-format] Align block comment decorations

2017-02-16 Thread Krasimir Georgiev via Phabricator via cfe-commits
krasimir updated this revision to Diff 88713. krasimir added a comment. - Align decorations of block comments spanning two lines https://reviews.llvm.org/D29943 Files: lib/Format/BreakableToken.cpp lib/Format/BreakableToken.h unittests/Format/FormatTest.cpp unittests/Format/FormatTestCo

[PATCH] D29943: [clang-format] Align block comment decorations

2017-02-16 Thread Manuel Klimek via Phabricator via cfe-commits
klimek accepted this revision. klimek added a comment. This revision is now accepted and ready to land. lg https://reviews.llvm.org/D29943 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-com

[PATCH] D29943: [clang-format] Align block comment decorations

2017-02-16 Thread Krasimir Georgiev via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL295312: [clang-format] Align block comment decorations (authored by krasimir). Changed prior to commit: https://reviews.llvm.org/D29943?vs=88713&id=88714#toc Repository: rL LLVM https://reviews.llvm

[PATCH] D29943: [clang-format] Align block comment decorations

2017-02-16 Thread Sylvestre Ledru via Phabricator via cfe-commits
sylvestre.ledru added a comment. Maybe this could be added to the release notes? Repository: rL LLVM https://reviews.llvm.org/D29943 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commit

Re: [PATCH] D29943: [clang-format] Align block comment decorations

2017-03-14 Thread Krasimir Georgiev via cfe-commits
Great! Thank you very much! There are two more major comment features that I added: comment reflowing and automatically adding namespace end comments, and now I know where to put the announcement about them! Cheers, Krasimir On Tue, Mar 14, 2017 at 10:56 AM, Sylvestre Ledru via Phabricator < revi

Re: [PATCH] D29943: [clang-format] Align block comment decorations

2017-03-14 Thread Sylvestre Ledru via cfe-commits
Welcome! Don't hesitate if you need anything else! Le 14/03/2017 à 10:59, Krasimir Georgiev a écrit : > Great! Thank you very much! There are two more major comment features that I > added: comment reflowing and automatically adding namespace end comments, and > now I know where to put the ann