Skip to content

Conversation

@jhinder
Copy link
Contributor

@jhinder jhinder commented Apr 29, 2023

Fixes #61343.

The lambda parameter's type was added to the array twice, causing its members to be considered twice too. This resulted in the wrong number of additional overloads being shown in the completion item's tooltip ($2n-1$ instead of $n-1$).
This fix makes the addition of the type symbol conditional so that it doesn't appear more than once in the array of type symbols.

@jhinder jhinder requested a review from a team as a code owner April 29, 2023 22:16
@ghost ghost added Community The pull request was submitted by a contributor who is not a Microsoft employee. Area-IDE untriaged Issues and PRs which have not yet been triaged by a lead labels Apr 29, 2023
@genlu genlu removed the untriaged Issues and PRs which have not yet been triaged by a lead label May 4, 2023
Copy link
Member

@genlu genlu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@genlu genlu merged commit a959bbd into dotnet:main May 4, 2023
@ghost ghost added this to the Next milestone May 4, 2023
@jhinder jhinder deleted the fix/61343-lambda-member-overloads branch May 4, 2023 20:45
@Cosifne Cosifne modified the milestones: Next, 17.7 P2 May 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area-IDE Community The pull request was submitted by a contributor who is not a Microsoft employee.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

IntelliSense/Auto-complete off-by-one on overload count on members of lambda argument objects

3 participants