owenpan added a comment. In D150403#4350325 <https://reviews.llvm.org/D150403#4350325>, @galenelias wrote:
> In D150403#4347323 <https://reviews.llvm.org/D150403#4347323>, > @HazardyKnusperkeks wrote: > >> We'll wait a bit, if someone might have a comment. And (at least I) need >> name and email for the commit. > > Name: Galen Elias > Email: galenelias at gmail dot com Can you put your authorship in the 'John Doe <j...@llvm.org>' <https://llvm.org/docs/Phabricator.html#committing-someone-s-change-from-phabricator> format? In D150403#4358562 <https://reviews.llvm.org/D150403#4358562>, @rymiel wrote: > This patch also fixes https://github.com/llvm/llvm-project/issues/52911 > (which is probably a duplicate anyway) @galenelias Can you update the summary? ================ Comment at: clang/unittests/Format/TokenAnnotatorTest.cpp:1789-1790 +TEST_F(TokenAnnotatorTest, UnderstandsNestedBlocks) { + // The closing braces are not annotated. It doesn't seem to cause a + // problem. So we only test for the opening braces. + auto Tokens = annotate("{\n" ---------------- CHANGES SINCE LAST ACTION https://reviews.llvm.org/D150403/new/ https://reviews.llvm.org/D150403 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits