[PATCH] D53025: [clang-tidy] implement new check for const return types.

Jonas Toth via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Oct 31 12:11:59 PDT 2018


JonasToth added a comment.

Thank you very much for contributing to LLVM/clang-tidy! More patches are always welcome ;)

I found one issue we have overseen that should be fixed. Would you be so kind and do that before comitting?



================
Comment at: test/clang-tidy/readability-const-return-type.cpp:6
+
+#include <type_traits>
+
----------------
One nit: system headers are not allowed in clang-tidy tests. We do mock the required content. Sorry that I overlooked that.

You can take the implementation from here: https://en.cppreference.com/w/cpp/types/add_cv


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D53025





More information about the cfe-commits mailing list