[clang-tools-extra] [clangd] Do not show `aParam` parameter hint for argument spelled `param` (PR #119162)
Nathan Ridge via cfe-commits
cfe-commits at lists.llvm.org
Mon Dec 9 22:55:12 PST 2024
HighCommander4 wrote:
> I have mixed feelings about this. The change is small, but it seems tied to a particular coding style, and the implementation appears to be "optimized" specifically for that style, which isn’t we do generally.
Would you feel any differently if the implementation was generalized to detect and ignore a one-character prefix from either operand of the comparison, so that e.g. `mFoo: kFoo` would be considered a match as well?
https://github.com/llvm/llvm-project/pull/119162
More information about the cfe-commits
mailing list