Version Used: Version 17.2.0 Preview 1.0
Steps to Reproduce:
- Create a local function like this:
void EnableSomething(bool enabled) {}
- Call it and turn on the
SuppressForParametersThatMatchMethodIntent flag.
Expected Behavior:
No parameter hint for the bool parameter.
Actual Behavior:
Parameter hint for the bool parameter.
This is because the code is filtering out anything but ordinary method kinds here: https://github.com/dotnet/roslyn/blob/main/src/Features/Core/Portable/InlineHints/AbstractInlineParameterNameHintsService.cs#L221-L223. I don't believe location functions should be excluded by this check.
Version Used: Version 17.2.0 Preview 1.0
Steps to Reproduce:
SuppressForParametersThatMatchMethodIntentflag.Expected Behavior:
No parameter hint for the bool parameter.
Actual Behavior:
Parameter hint for the bool parameter.
This is because the code is filtering out anything but ordinary method kinds here: https://github.com/dotnet/roslyn/blob/main/src/Features/Core/Portable/InlineHints/AbstractInlineParameterNameHintsService.cs#L221-L223. I don't believe location functions should be excluded by this check.