[PATCH] D28768: [clang-tidy] Add check 'modernize-return-braced-init-list'

[Aaron Ballman via Phabricator via cfe-commits]

aaron.ballman added inline comments.


================
Comment at: clang-tidy/modernize/ReturnBracedInitListCheck.cpp:59
+  // Skip explicit construcotrs.
+  if (MatchedConstructExpr->getConstructor()->isExplicit())
+    return;
----------------
Can't this be handled as part of the matcher? Something like `hasDeclaration(cxxConstructorDecl(isExplicit()))` should work.


Repository:
  rL LLVM

https://reviews.llvm.org/D28768