Skip to content

Revert "Analyze flow in receiver of extension method group in delegate creation"#60087

Merged
333fred merged 1 commit intomainfrom
revert-59874-bad-region
Mar 10, 2022
Merged

Revert "Analyze flow in receiver of extension method group in delegate creation"#60087
333fred merged 1 commit intomainfrom
revert-59874-bad-region

Conversation

@333fred
Copy link
Copy Markdown
Member

@333fred 333fred commented Mar 10, 2022

Reverts #59874 to unblock insertion.

@333fred 333fred requested a review from a team as a code owner March 10, 2022 18:16
@ghost ghost added the Area-Compilers label Mar 10, 2022
@333fred
Copy link
Copy Markdown
Member Author

333fred commented Mar 10, 2022

FYI @jcouv @RikkiGibson

@333fred 333fred enabled auto-merge (squash) March 10, 2022 18:18
@RikkiGibson
Copy link
Copy Markdown
Member

Is it the case that when accessing a static member from an instance member, 'this' is implicitly used as the receiver?

@333fred
Copy link
Copy Markdown
Member Author

333fred commented Mar 10, 2022

Is it the case that when accessing a static member from an instance member, 'this' is implicitly used as the receiver?

Yes, there's a BoundThisReference as a receiver for the BoundMethodGroup.

@333fred 333fred disabled auto-merge March 10, 2022 19:41
@333fred 333fred merged commit bcec44a into main Mar 10, 2022
@ghost ghost added this to the Next milestone Mar 10, 2022
@333fred
Copy link
Copy Markdown
Member Author

333fred commented Mar 10, 2022

Unit tests and correctness passed. Merged to unblock the insertion.

@333fred 333fred deleted the revert-59874-bad-region branch March 10, 2022 19:42
@allisonchou allisonchou modified the milestones: Next, 17.2.P3 Mar 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants