Skip to content
This repository was archived by the owner on Jan 23, 2023. It is now read-only.

Moving the Utf8Parser/Utf8Formatter to be shared#33348

Merged
stephentoub merged 1 commit intodotnet:masterfrom
tannergooding:share-utf8-parseformat
Nov 9, 2018
Merged

Moving the Utf8Parser/Utf8Formatter to be shared#33348
stephentoub merged 1 commit intodotnet:masterfrom
tannergooding:share-utf8-parseformat

Conversation

@tannergooding
Copy link
Copy Markdown
Member

As per #20873 (comment), I split the PR into multiple

This is the second PR and moves the UTF8 Parser and Formatter to be shared. This will make tracking the history better and make the PR that moves it into S.P.CoreLib to be smaller.

@tannergooding
Copy link
Copy Markdown
Member Author

CC. @stephentoub, @jkotas

@tannergooding
Copy link
Copy Markdown
Member Author

Also CC. @GrabYourPitchforks

Copy link
Copy Markdown
Member

@stephentoub stephentoub left a comment

Choose a reason for hiding this comment

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

Thanks.

@stephentoub stephentoub merged commit 9c68db7 into dotnet:master Nov 9, 2018
dotnet-maestro-bot pushed a commit to dotnet-maestro-bot/corert that referenced this pull request Nov 9, 2018
Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>
dotnet-maestro-bot pushed a commit to dotnet-maestro-bot/coreclr that referenced this pull request Nov 9, 2018
Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>
jkotas pushed a commit to dotnet/coreclr that referenced this pull request Nov 9, 2018
Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>
jkotas pushed a commit to dotnet/corert that referenced this pull request Nov 9, 2018
Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>
@karelz karelz added this to the 3.0 milestone Nov 15, 2018
picenka21 pushed a commit to picenka21/runtime that referenced this pull request Feb 18, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants