[PATCH] D28768: [clang-tidy] Add check 'modernize-return-braced-init-list'
Jonas Devlieghere via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Jan 17 10:57:14 PST 2017
JDevlieghere updated this revision to Diff 84700.
JDevlieghere added a comment.
- Added test cases suggested by @Prazek
- Added test cases suggested by @alexfh
I don't match on `CXXUnresolvedConstructExpr` so the template dependent cases are not impacted by this check. This is what I intended, because we cannot make assumptions about the constructor being explicit, narrowing, etc.
Repository:
rL LLVM
https://reviews.llvm.org/D28768
Files:
clang-tidy/modernize/CMakeLists.txt
clang-tidy/modernize/ModernizeTidyModule.cpp
clang-tidy/modernize/ReturnBracedInitListCheck.cpp
clang-tidy/modernize/ReturnBracedInitListCheck.h
docs/ReleaseNotes.rst
docs/clang-tidy/checks/list.rst
docs/clang-tidy/checks/modernize-return-braced-init-list.rst
test/clang-tidy/modernize-return-braced-init-list.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D28768.84700.patch
Type: text/x-patch
Size: 12793 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20170117/eadc611c/attachment-0001.bin>
More information about the cfe-commits
mailing list