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

Support non-allocating view on string properties values of JSON in System.Text.Json.JsonElement #78885

Closed
tomas-pajurek opened this issue Nov 27, 2022 · 2 comments
Closed

Support non-allocating view on string properties values of JSON in System.Text.Json.JsonElement #78885

tomas-pajurek opened this issue Nov 27, 2022 · 2 comments
Labels
area-System.Text.Json

Comments

@tomas-pajurek
Copy link

In .NET 7, there is a new set of methods CopyString(...) on the Utf8JsonReader that allows for non-allocating view of string properties. Related issue: #54410.

With the same background and motivation as in the related issue, would it make sense to introduce similar change also to JsonElement type?
@ghost ghost added the untriaged New issue has not been triaged by the area owner label Nov 27, 2022
@ghost
Copy link

ghost commented Nov 27, 2022

Tagging subscribers to this area: @dotnet/area-system-text-json, @gregsdennis
See info in area-owners.md if you want to be subscribed.

Issue Details

In .NET 7, there is a new set of methods CopyString(...) on the Utf8JsonReader that allows for non-allocating view of string properties. Related issue: #54410.

With the same background and motivation as in the related issue, would it make sense to introduce similar change also to JsonElement type?

Author: tomas-pajurek
Assignees: -
Labels:

area-System.Text.Json
Milestone: -

@eiriktsarpalis
Copy link
Member

Duplicate of #77666 (comment)

@eiriktsarpalis eiriktsarpalis marked this as a duplicate of #77666 Nov 27, 2022
@ghost ghost removed the untriaged New issue has not been triaged by the area owner label Nov 27, 2022
@ghost ghost locked as resolved and limited conversation to collaborators Dec 27, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
area-System.Text.Json
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@eiriktsarpalis @tomas-pajurek