diff options
| author | Nathan James <n.james93@hotmail.co.uk> | 2020-03-22 19:09:00 +0000 |
|---|---|---|
| committer | Nathan James <n.james93@hotmail.co.uk> | 2020-03-23 13:45:34 +0000 |
| commit | 7693a9b9314083eafd9b5b1e19b02aac06962eb2 (patch) | |
| tree | ea87ba559920eca03b83130cff465be87c94025d /lldb/source/Plugins/ScriptInterpreter/Python/ScriptInterpreterPython.h | |
| parent | 78e2a3c678463caeec1524baa96cdeb6fcdb48be (diff) | |
| download | llvm-7693a9b9314083eafd9b5b1e19b02aac06962eb2.zip llvm-7693a9b9314083eafd9b5b1e19b02aac06962eb2.tar.gz llvm-7693a9b9314083eafd9b5b1e19b02aac06962eb2.tar.bz2 | |
[clang-tidy] Fix RenamerClangTidy handling qualified TypeLocs
Summary:
Previously if a type was accessed with a qualifier, RenamerClangTidy wouldn't rename the TypeLoc, this patch addresses this shortfall by trying to find the Unqualified TypeLoc first. Also fixed a broken test case that was dependent on this broken behaviour.
Example:
```
struct a{};
void foo(const a&);
void foo(a&);
void foo(a);
void foo(a&&);
void foo(const a);
```
exec `-checks=readability-identifier-naming --config="{CheckOptions: [{key: readability-identifier-naming.StructCase, value: CamelCase}]}" -fix`
Current Behaviour:
```
struct A{};
void foo(const a&);
void foo(A&);
void foo(A);
void foo(A&&);
void foo(const a);
```
Proposed new behaviour:
```
struct A{};
void foo(const A&);
void foo(A&);
void foo(A);
void foo(A&&);
void foo(const A);
```
Reviewers: aaron.ballman, gribozavr2, alexfh
Reviewed By: aaron.ballman
Subscribers: xazax.hun, cfe-commits
Tags: #clang
Differential Revision: https://reviews.llvm.org/D76549
Diffstat (limited to 'lldb/source/Plugins/ScriptInterpreter/Python/ScriptInterpreterPython.h')
0 files changed, 0 insertions, 0 deletions
