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.

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

[release/5.0-preview8] Add methods to convert between hexadecimal strings and bytes (#37546) #39658

Merged
merged 1 commit into from
Jul 21, 2020

Conversation

jeffhandley
Copy link
Member

Port of #37546 into the release/5.0-preview8 branch.

Description

This adds two methods (with overloads) ToHexString and FromHexString to System.Convert. Across the libraries, one can find various different classes and methods handling hex in more or less creative ways. This PR moves a lot of those to the implementations to HexConverter.

Customer Impact

This was a community-submitted PR that adds new (already-approved) APIs that we want to ship in .NET 5.0.

Add methods to convert between hexadecimal strings and bytes, and converts internal usage to use those APIs when possible, or reuse the shared code as helpers otherwise.
@jeffhandley jeffhandley added this to the 5.0.0 Preview 8 milestone Jul 20, 2020
@jeffhandley jeffhandley requested a review from bartonjs July 20, 2020 21:43
@Dotnet-GitSync-Bot
Copy link
Collaborator

Note regarding the new-api-needs-documentation label:

This serves as a reminder for when your PR is modifying a ref *.cs file and adding/modifying public APIs, to please make sure the API implementation in the src *.cs file is documented with triple slash comments, so the PR reviewers can sign off that change.

@Dotnet-GitSync-Bot
Copy link
Collaborator

I couldn't figure out the best area label to add to this PR. If you have write-permissions please help me learn by adding exactly one area label.

@jeffhandley jeffhandley merged commit 3fb6653 into dotnet:release/5.0-preview8 Jul 21, 2020
@jeffhandley jeffhandley deleted the hexadecimal-37546 branch July 21, 2020 00:40
@ghost ghost locked as resolved and limited conversation to collaborators Dec 8, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants