[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