[PATCH] D38284: [clang-tidy] Fix google-readability-namespace-comments handling of C++17 nested namespaces

Alexander Kornienko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Oct 12 07:25:58 PDT 2017


alexfh added inline comments.


================
Comment at: clang-tidy/readability/NamespaceCommentCheck.cpp:97
+  }
+  while (Lexer::getRawToken(LBracketLocation, Tok, Sources, getLangOpts()) ||
+         !Tok.is(tok::l_brace)) {
----------------
alexfh wrote:
> The check started triggering an assertion failure and incredible slowness (infinite loops?) on some real files. I've not yet come up with an isolated test case, but all this seems to be happening around this loop.
> 
> I'm going to revert this revision. A bug report (and hopefully a test case ;) will follow.
> 
Reverted in r315580.


https://reviews.llvm.org/D38284





More information about the cfe-commits mailing list