[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