[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