[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 01:11:42 PST 2017
JDevlieghere added a comment.
In https://reviews.llvm.org/D28768#647949, @Prazek wrote:
> Do you have some results from running it on LLVM? If nothing crashes and all fixit are correct then LGTM.
There's still an issue with type narrowing, which is allowed in regular constructor but not in braced initializer lists. I'd like to skip cases where narrowing occurs, either in the matcher or in the check method, but I haven't figured out how yet.
More information about the cfe-commits