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

[Aaron Ballman via Phabricator via cfe-commits]

aaron.ballman accepted this revision.
aaron.ballman added a comment.

There's one small wording nit with the diagnostic, but once that's resolved, LGTM as well.



================
Comment at: clang-tidy/modernize/ReturnBracedInitListCheck.cpp:65
+
+  auto Diag = diag(Loc, "to avoid repeating the return type from the "
+                        "declaration; use a braced initializer list instead");
----------------
This diagnostic reads strangely. I think you should drop the "to" at the start of the sentence.


Repository:
  rL LLVM

https://reviews.llvm.org/D28768