diff options
author | Nathan <n.james93@hotmail.co.uk> | 2020-01-27 23:46:53 +0000 |
---|---|---|
committer | Nathan <n.james93@hotmail.co.uk> | 2020-01-27 23:47:51 +0000 |
commit | c3d20fd472000c7abdc2c9cad051a2ab339709d9 (patch) | |
tree | e0b05c8a29d3619ad763c79684eb0818d20a67b2 /clang/lib/Frontend/CompilerInvocation.cpp | |
parent | 9521c18438a9f09663f3dc68aa7581371c0653c9 (diff) | |
download | llvm-c3d20fd472000c7abdc2c9cad051a2ab339709d9.zip llvm-c3d20fd472000c7abdc2c9cad051a2ab339709d9.tar.gz llvm-c3d20fd472000c7abdc2c9cad051a2ab339709d9.tar.bz2 |
[clang-tidy] readability-identifier-naming disregards parameters restrictions on main like functions
Summary:
Typically most main functions have the signature:
```
int main(int argc, char *argv[])
```
To stick with convention when renaming parameters we should ignore the `argc` and `argv` names even if the parameter style says they should be renamed. This patch addresses this by checking all ParmVarDecls if they form part of a function with a signature that matches main `int name(int argc, char * argv[], (optional char *env[]))`
Reviewers: aaron.ballman, JonasToth, alexfh, hokein
Reviewed By: aaron.ballman
Subscribers: Mordante, merge_guards_bot, xazax.hun, kristof.beyls, cfe-commits
Tags: #clang, #clang-tools-extra
Differential Revision: https://reviews.llvm.org/D73098
Diffstat (limited to 'clang/lib/Frontend/CompilerInvocation.cpp')
0 files changed, 0 insertions, 0 deletions