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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to