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

Jonas Toth via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Oct 30 13:48:48 PDT 2018


JonasToth added a comment.

Am 30.10.18 um 21:47 schrieb Aaron Ballman via Phabricator:

> aaron.ballman added a comment.
> 
> I think this is getting really close! One question: would it make more sense to name this `readability-const-type-return` or `readability-const-return-type` instead? It's not that the functions return a const *value* that's the issue, it's that the declared return type is top-level const. I think removing "value" and using "type" instead would be an improvement (and similarly, rename the files and the check).

There is a `rename-check.py` script in the repository. Just to prevent a
lot of manual work ;)


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D53025





More information about the cfe-commits mailing list