| Issue |
165346
|
| Summary |
[clang] Rename some variables with stale names
|
| Labels |
good first issue,
clang:frontend,
code-cleanup
|
| Assignees |
|
| Reporter |
localspook
|
Commit bcd0350 renamed `DeclaratorInfo` to `TypeSourceInfo`. This left behind some cases like:
https://github.com/llvm/llvm-project/blob/66b481556e01e6e2508d7c9146849167b9e0323f/clang/lib/Sema/SemaTemplate.cpp#L952
Where the variable name is now meaningless. I was quite confused the first time I stumbled into one of these. These variables should be renamed; something simple like replacing `DI` with `TSI` will work just fine.
Doing a
```sh
git grep -n 'TypeSourceInfo.*DI'
```
should find most of them (I see 33 hits running it locally).
Note: if you want to take on this issue, it's a good idea to post a comment saying you're looking into it, but you don't need to wait for anyone's approval to get started; just go for it!
![]()
_______________________________________________
llvm-bugs mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-bugs
