[PATCH] D56160: [clang-tidy] modernize-use-trailing-return check

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jan 2 08:09:56 PST 2019


aaron.ballman added inline comments.


================
Comment at: test/clang-tidy/modernize-use-trailing-return.cpp:2
+// RUN: %check_clang_tidy %s modernize-use-trailing-return %t -- -- --std=c++14
+
+namespace std {
----------------
lebedev.ri wrote:
> Missing test coverage:
> * macros
> * is there tests for implicit functions?
> * Out-of-line function with body.
Also:
  * functions with attributes in the type position `int f() [[]];`
  * functions without attributes in the type position `[[]] int f();` and `int f [[]] ();`
  * lambdas?
  * Special functions without return types, like constructors and destructors
  * Conversion operators. `struct S {  operator int() const; };`


Repository:
  rCTE Clang Tools Extra

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D56160/new/

https://reviews.llvm.org/D56160





More information about the cfe-commits mailing list