Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Suppression operator on L-values #27522

Closed
jcouv opened this issue Jun 6, 2018 · 0 comments
Closed

Suppression operator on L-values #27522

jcouv opened this issue Jun 6, 2018 · 0 comments
Assignees
@jcouv
Labels
4 - In Review A fix for the issue is submitted for review. Area-Compilers Area-Language Design New Language Feature - Nullable Reference Types Nullable Reference Types
Milestone
16.1.P1

Comments

@jcouv
Copy link
Member

jcouv commented Jun 6, 2018 •
edited
Loading

  • M(out x!); (note this also definitely assigns to x through the !)
  • M(out (x!));
  • M(out (RefReturning()!));
  • x! = y;
  • M(out string x!);

Current implementation is to allow in out scenarios, but disallow in assignment scenarios.
@jcouv jcouv added this to the 16.0 milestone Jun 6, 2018
@jcouv jcouv mentioned this issue Jun 6, 2018
Merged
@cston cston mentioned this issue Jul 9, 2018
Closed
This was referenced Sep 17, 2018
Closed
Closed
Closed
@jcouv jcouv modified the milestones: 16.0, 16.0.P2 Nov 9, 2018
@gafter gafter modified the milestones: 16.0.P2, 16.0.P3 Jan 8, 2019
@jaredpar jaredpar mentioned this issue Jan 24, 2019
Closed
@cston cston mentioned this issue Feb 6, 2019
Open
@jcouv jcouv added 4 - In Review A fix for the issue is submitted for review. 3 - Working and removed 4 - In Review A fix for the issue is submitted for review. labels Mar 3, 2019
@jcouv jcouv mentioned this issue Mar 5, 2019
Merged
@jcouv jcouv added 4 - In Review A fix for the issue is submitted for review. and removed 3 - Working labels Mar 5, 2019
agocke added a commit to agocke/roslyn that referenced this issue Mar 5, 2019
@agocke
Regression test for suppression on ref-returns
fe3ddbc 
Verifies that the current implementation matches the latest
LDM decisions on nullable suppression and out variables

Closes dotnet#27522
@agocke agocke mentioned this issue Mar 5, 2019
Merged
agocke added a commit that referenced this issue Mar 5, 2019
@agocke
Regression test for suppression on ref-returns (#33870)
80a7fac 
Verifies that the current implementation matches the latest
LDM decisions on nullable suppression and out variables

Closes #27522
@jcouv jcouv modified the milestones: 16.0.P3, 16.1.P1 Mar 7, 2019
@RikkiGibson RikkiGibson mentioned this issue Jun 20, 2019
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jcouv jcouv

Labels
4 - In Review A fix for the issue is submitted for review. Area-Compilers Area-Language Design New Language Feature - Nullable Reference Types Nullable Reference Types
Projects
None yet
Milestone
16.1.P1
Development

No branches or pull requests
2 participants
@gafter @jcouv