Skip to content

Add missing delegate casts#58170

Merged
tmat merged 1 commit intodotnet:mainfrom
tmat:FixSymbolSpecificationDialog
Dec 7, 2021
Merged

Add missing delegate casts#58170
tmat merged 1 commit intodotnet:mainfrom
tmat:FixSymbolSpecificationDialog

Conversation

@tmat
Copy link
Copy Markdown
Member

@tmat tmat commented Dec 7, 2021

The casts were incorrectly removed in 4ef3640

The IDE incorrectly reports them as unnecessary (filed #58171).
image

Fixes https://devdiv.visualstudio.com/DevDiv/_workitems/edit/1434559

@tmat tmat requested a review from a team as a code owner December 7, 2021 19:49
@tmat
Copy link
Copy Markdown
Member Author

tmat commented Dec 7, 2021

@CyrusNajmabadi PTAL

@tmat tmat enabled auto-merge (squash) December 7, 2021 20:11
@tmat tmat merged commit 9d335cd into dotnet:main Dec 7, 2021
@ghost ghost added this to the Next milestone Dec 7, 2021
@jmarolf
Copy link
Copy Markdown
Contributor

jmarolf commented Dec 8, 2021

Thanks @tmat! Just ran into this locally

@tmat tmat deleted the FixSymbolSpecificationDialog branch December 8, 2021 03:01
333fred added a commit to 333fred/roslyn that referenced this pull request Dec 10, 2021
…rations

* upstream/main: (396 commits)
  Update several ExpressionCompiler unit tests for inferred delegate types (dotnet#58203)
  Avoid calculating inferred delegate type unnecessarily in conversions and type inference (dotnet#58115)
  OmniSharp options (dotnet#58208)
  Fix generator caching in compiler server (dotnet#57177)
  Clarify use of null as an initialized value
  BoundDecisionDag.Rewrite - Avoid capturing the replacement map (dotnet#58137)
  Address feedback
  Add regex parser tests
  Add additional test for another bug report
  Add additional test for another bug report
  Add additional test for another bug report
  Add additional test for another bug report
  Add additional test for another bug report
  Add missing delegate casts (dotnet#58170)
  Add additional test for another bug report
  Add additional test for another bug report
  Add additional test for another bug report
  Add additional test for another bug report
  Add additional test for another bug report
  Add additional test for another bug report
  ...
@Cosifne Cosifne modified the milestones: Next, 17.1.P3 Jan 5, 2022
@ryzngard ryzngard added UX Review Not Required UX Review Not Required and removed Needs UX Triage labels Apr 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants