Visitar URL original
Use 3 parameter Encoding.GetString for default string values by ObsidianMinor · Pull Request #6828 · protocolbuffers/protobuf · GitHub
Skip to content

Use 3 parameter Encoding.GetString for default string values - #6828

Merged
jtattermusch merged 1 commit into
protocolbuffers:masterfrom
ObsidianMinor:csharp/get-string-overload
Nov 5, 2019
Merged

jtattermusch merged 1 commit into
protocolbuffers:masterfrom
ObsidianMinor:csharp/get-string-overload

Conversation

@ObsidianMinor

Copy link
Copy Markdown
Contributor

Encoding.GetString(byte[]) is actually incompatible with .NET Standard 1.0.

@ObsidianMinor

Copy link
Copy Markdown
Contributor Author

The new .cs files from #6824 (OldExtensions2.cs and OldExtensions1.cs) didn't get added to Makefile.am.

@ObsidianMinor
ObsidianMinor force-pushed the csharp/get-string-overload branch from de85379 to e119a43 Compare November 4, 2019 18:31
@ObsidianMinor

Copy link
Copy Markdown
Contributor Author

Rebased on master, try rerunning tests @jtattermusch

@jtattermusch jtattermusch left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM.

@jtattermusch
jtattermusch merged commit d052a05 into protocolbuffers:master Nov 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants