[PATCH] D144510: [clang-tidy] improve readability-identifier-naming hungarian options test
Douglas Chen via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Apr 10 01:37:32 PDT 2023
dougpuob added a comment.
Herald added a subscriber: PiotrZSL.
Hi @amurzeau
I missed the review before landing. I have a suggestion regarding the customized prefix "cust" because it confused me when I found the Hungarian notation patterns in the output message. Perhaps using "my" instead of "cust" would improve readability.
Take some examples:
- const char *`custszNamePtr` = "Name"; --> `myszNamePtr`
- uint8_t `custu8ValueU8` = 0; --> `myu8ValueU8`
- DWORD `custdwMsDword` = 0; --> `mydwMsDword`
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D144510/new/
https://reviews.llvm.org/D144510
More information about the cfe-commits
mailing list