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

[cdac] Implement basic data stream reader and writer in C #99300

Closed
Tracked by #99298
lambdageek opened this issue Mar 5, 2024 · 3 comments
Closed
Tracked by #99298

[cdac] Implement basic data stream reader and writer in C #99300

lambdageek opened this issue Mar 5, 2024 · 3 comments
Labels
area-Diagnostics-coreclr enhancement Product code improvement that does NOT require public API changes/additions
Milestone
9.0.0

Comments

@lambdageek
Copy link
Member

lambdageek commented Mar 5, 2024 •
edited
Loading

Using the spec from #99299 implement a basic data stream writer and reader.

There's a couple of parts here:

  • A basic reader/writer in standard C without the PAL in src/native so that we can reuse it for NativeAOT and Mono.
  • A convenience C++ interface in coreclr.
  • Initial data stream creation and basic type registration in coreclr
@lambdageek lambdageek mentioned this issue Mar 5, 2024
Open
35 tasks
@ghost ghost added the untriaged New issue has not been triaged by the area owner label Mar 5, 2024
@ghost
Copy link

ghost commented Mar 5, 2024

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

Issue Details

null

Author: lambdageek
Assignees: -
Labels:

area-System.IO
Milestone: -

@lambdageek lambdageek mentioned this issue Mar 5, 2024
Closed
@ghost
Copy link

ghost commented Mar 5, 2024

Tagging subscribers to this area: @tommcdon
See info in area-owners.md if you want to be subscribed.

Issue Details

Using the spec from #99299 implement a basic data stream writer and reader.

There's a couple of parts here:

  • A basic reader/writer in standard C without the PAL in src/native so that we can reuse it for NativeAOT and Mono.
  • A convenience C++ interface in coreclr.
  • Initial data stream creation and basic type registration in coreclr
Author: lambdageek
Assignees: -
Labels:

area-System.IO, area-Diagnostics-coreclr, untriaged
Milestone: -

@lambdageek lambdageek removed area-System.IO untriaged New issue has not been triaged by the area owner labels Mar 5, 2024
@lambdageek lambdageek added this to the 9.0.0 milestone Mar 5, 2024
@lambdageek lambdageek mentioned this issue Mar 8, 2024
Closed
@tommcdon tommcdon added the enhancement Product code improvement that does NOT require public API changes/additions label Mar 11, 2024
@lambdageek
Copy link
Member Author

We went with a JSON contract descriptor in #100365, so a C reader no longer makes sense for a NativeAOT-based reader like in #100623

@github-actions github-actions bot locked and limited conversation to collaborators May 5, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
area-Diagnostics-coreclr enhancement Product code improvement that does NOT require public API changes/additions
Projects
None yet
Milestone
9.0.0
Development

Successfully merging a pull request may close this issue.

[cdac] Add data stream library elinor-fung/runtime
2 participants
@lambdageek @tommcdon