[PATCH] D34330: [clang-format] handle `if constexpr`

2017-06-19 Thread Daniel Jasper via Phabricator via cfe-commits
djasper closed this revision. djasper added a comment. Yes, I saw. As this version seems to handle the one-line case correctly, I submitted this one as r305666. https://reviews.llvm.org/D34330 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D34330: [clang-format] handle `if constexpr`

2017-06-18 Thread Jacob Bandes-Storch via Phabricator via cfe-commits
jtbandes added a comment. Thanks for the review. Please note that there was a prior effort to implement this in https://reviews.llvm.org/D26953. However if you are happy with this version, feel free to commit (as I don’t have commit access). https://reviews.llvm.org/D34330 _

[PATCH] D34330: [clang-format] handle `if constexpr`

2017-06-18 Thread Daniel Jasper via Phabricator via cfe-commits
djasper accepted this revision. djasper added a comment. This revision is now accepted and ready to land. Looks good. Thanks for implementing this. https://reviews.llvm.org/D34330 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.

[PATCH] D34330: [clang-format] handle `if constexpr`

2017-06-18 Thread Jacob Bandes-Storch via Phabricator via cfe-commits
jtbandes created this revision. Herald added a subscriber: klimek. Changes to handle `if constexpr` the same way as `if`. https://reviews.llvm.org/D34330 Files: lib/Format/ContinuationIndenter.cpp lib/Format/TokenAnnotator.cpp lib/Format/UnwrappedLineParser.cpp unittests/Format/FormatTe