Skip to content

Add support for named ref/out arguments#49

Merged
DustinCampbell merged 1 commit intodotnet:masterfrom
kiootic:named-ref-out
Mar 25, 2017
Merged

Add support for named ref/out arguments#49
DustinCampbell merged 1 commit intodotnet:masterfrom
kiootic:named-ref-out

Conversation

@kiootic
Copy link
Contributor

@kiootic kiootic commented Mar 15, 2017

As title. Test case:

public class Test {
    void Out(out int a) => a = 0;
    void Ref(ref int a) => a = 0;
    public void M() {
        int q;
        Out(a: out q);
        Ref(a: ref q);
    }
}

@dnfclas
Copy link

dnfclas commented Mar 15, 2017

@kiootic,
Thanks for your contribution.
To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla2.dotnetfoundation.org.

It will cover your contributions to all .NET Foundation-managed open source projects.
Thanks,
.NET Foundation Pull Request Bot

@DustinCampbell
Copy link
Member

DustinCampbell commented Mar 24, 2017

@kiootic: Not that you must sign the CLA as specified above before we can take this change. Are you able to do that?

@dnfclas
Copy link

dnfclas commented Mar 24, 2017

@kiootic, thanks for signing the contribution license agreement. We will now validate the agreement and then the pull request.

Thanks, .NET Foundation Pull Request Bot

@kiootic
Copy link
Contributor Author

kiootic commented Mar 24, 2017

Sorry about that, I thought to have it reviewed before signing it. I had signed the CLA now.

@DustinCampbell DustinCampbell merged commit 4d0e50c into dotnet:master Mar 25, 2017
@DustinCampbell DustinCampbell mentioned this pull request Mar 25, 2017
Merged
@kiootic kiootic deleted the named-ref-out branch March 31, 2017 14:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

cla-signed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@kiootic @dnfclas @DustinCampbell