Skip to content

C#: Fix data flow for out/ref parameters#1685

Merged
calumgrant merged 2 commits intogithub:masterfrom
hvitved:csharp/dataflow/out-flow-fix
Aug 6, 2019
Merged

C#: Fix data flow for out/ref parameters#1685
calumgrant merged 2 commits intogithub:masterfrom
hvitved:csharp/dataflow/out-flow-fix

Conversation

@hvitved
Copy link
Contributor

@hvitved hvitved commented Aug 2, 2019 •
edited
Loading

This PR fixes data flow out of out/ref parameters for methods with multiple out/ref parameters.

@hvitved hvitved added the C# label Aug 2, 2019
@hvitved hvitved marked this pull request as ready for review August 6, 2019 07:27
@hvitved hvitved requested a review from a team as a code owner August 6, 2019 07:27
@hvitved hvitved requested a review from calumgrant August 6, 2019 07:28
calumgrant
calumgrant approved these changes Aug 6, 2019
View reviewed changes
Copy link
Contributor

@calumgrant calumgrant left a comment

Choose a reason for hiding this comment

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

Good catch - LGTM.

@calumgrant calumgrant merged commit 2df0509 into github:master Aug 6, 2019
@hvitved hvitved deleted the csharp/dataflow/out-flow-fix branch August 6, 2019 08:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@calumgrant calumgrant calumgrant approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

C#

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@hvitved @calumgrant