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

Alexandru Octavian Buțiu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Sep 29 12:01:50 PDT 2017


predator5047 updated this revision to Diff 117193.
predator5047 marked 12 inline comments as done.
predator5047 added a comment.

Address review comments:

- Don't use auto if the type is not spelled out in initialization.
- Better names for variables.
- Elide braces.
- Format changes.




https://reviews.llvm.org/D38284

Files:
  clang-tidy/readability/NamespaceCommentCheck.cpp
  clang-tidy/readability/NamespaceCommentCheck.h
  test/clang-tidy/google-readability-nested-namespace-comments.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D38284.117193.patch
Type: text/x-patch
Size: 6040 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20170929/c5209c66/attachment.bin>


More information about the cfe-commits mailing list