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

Remove point #1585

Merged
merged 1 commit into from
Aug 8, 2019
Merged

Remove point #1585

merged 1 commit into from
Aug 8, 2019

Conversation

hughbe
Copy link
Contributor

@hughbe hughbe commented Aug 7, 2019 •
edited by ghost
Loading

Proposed changes

  • Remove POINT preferring Point which marshals the same

Customer Impact

  • Reduced allocations as we now use classes instead of structs

Regression?

  • No

Risk

  • Minimal

Test methodology

Extracted from #1508

Microsoft Reviewers: Open in CodeFlow

@hughbe hughbe requested a review from a team as a code owner August 7, 2019 19:08
@RussKie RussKie merged commit 8e30dba into dotnet:master Aug 8, 2019
@ghost ghost added this to the 3.0.0-Preview9 milestone Aug 8, 2019
@ghost ghost added the tell-mode label Aug 8, 2019
@hughbe hughbe deleted the remove-POINT branch August 8, 2019 10:43
@ghost ghost locked as resolved and limited conversation to collaborators Feb 5, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@RussKie RussKie RussKie approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
3.0-Preview9
Development

Successfully merging this pull request may close these issues.
2 participants
@hughbe @RussKie