Skip to content

Conversation

@cston
Copy link
Contributor

@cston cston commented Oct 12, 2022

No description provided.

@cston cston requested a review from a team as a code owner October 12, 2022 23:23
@ghost ghost added the Area-Compilers label Oct 12, 2022
// 2. e1 = ref e2: where e1 is a ref local or ref parameter then e2 must have a safe-to-escape equal to safe-to-escape for e1 and
// e2 must have ref-safe-to-escape at least as large as ref-safe-to-escape of the ref-safe-to-escape of e1
// https://github.com/dotnet/csharplang/blob/main/proposals/csharp-11.0/low-level-struct-improvements.md#rules-ref-reassignment
// For a ref reassignment in the form `e1 = ref e2` both of the following must be true:
Copy link
Member

Choose a reason for hiding this comment

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

Thanks! :-)

Copy link
Contributor

Choose a reason for hiding this comment

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

I think that permalinking the commit (y hotkey) will fix the issue of the file being moved.

You can then navigate to the file history from that permalink.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks @Therzok for the suggestion. I'll keep the links as is though, so the links refer to the latest text.

Copy link
Member

@jcouv jcouv left a comment

Choose a reason for hiding this comment

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

LGTM Thanks (iteration 1)

@jcouv jcouv self-assigned this Oct 13, 2022
@cston cston merged commit 19a9348 into dotnet:main Oct 13, 2022
@cston cston deleted the spec-refs branch October 13, 2022 02:31
@ghost ghost added this to the Next milestone Oct 13, 2022
@RikkiGibson RikkiGibson modified the milestones: Next, 17.5 P1 Oct 24, 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