[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