There are barely any lines I haven't changed in these files, so I think I could might as well leave it in an LLVM coding style conforming state. I also renamed 2 functions and moved addDependency out of line to ease on followup patches.
Details
Details
- Reviewers
NoQ baloghadamsoftware xazax.hun rnkovacs - Commits
- rGb9bc7ec304de: [analyzer][NFC] Use capital variable names, move methods out-of-line, rename…
rC358676: [analyzer][NFC] Use capital variable names, move methods out-of-line, rename…
rL358676: [analyzer][NFC] Use capital variable names, move methods out-of-line, rename…
Diff Detail
Diff Detail
- Repository
- rL LLVM
Event Timeline
Comment Actions
I did not check the patch yet but wanted to point out that we might not want to rush about renaming all the variables until the community decides on the coding guideline, see https://reviews.llvm.org/D59251
Comment Actions
Good point, sorry about that.
I wonder why this wasn't posted on cfe-dev. In any case, most of the variables are capitalized, there still is a consistency argument. Refactoring all the dependent patches would also be a pain in the buttocks for little gain.
Comment Actions
I agree that we shouldn't try real hard to fix our code until we have a clear "party line" on how exactly do we name our variables (and ideally also how do we invalidate our caches), but having variables and fields consistent with each other makes it look much better imo, so why not.