Skip to content

New io - #98

Open
alexkazik wants to merge 2 commits into
rafalh:masterfrom
alexkazik:new-io
Open

New io#98
alexkazik wants to merge 2 commits into
rafalh:masterfrom
alexkazik:new-io

Conversation

@alexkazik

@alexkazik alexkazik commented Aug 16, 2024

Copy link
Copy Markdown

I couldn't sit still after thinking about it, here is a first draft for a Read/Write which is always exact, and the "up to" version is only used to read/write files (and DirRawStream internally).

All tests run but it's untested otherwise.

It's just a suggestion.

I can squash/remove commits/MR if unwanted.

Edit: I forgot the latest changes and force pushed it.

@rafalh

rafalh commented Aug 22, 2026

Copy link
Copy Markdown
Owner

I don't really see a point in splitting those traits. I wanted to stay as close as possible to std::io API and there is no ReadFile and WriteFile API there. What exactly was the intention here? To only implement non-file traits on devices?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants