Skip to content

feat(io): impl Writer for std::io::Cursor#280

Merged
cpubot merged 1 commit intomasterfrom
std-io-cursor-write
Mar 30, 2026
Merged

feat(io): impl Writer for std::io::Cursor#280
cpubot merged 1 commit intomasterfrom
std-io-cursor-write

Conversation

@cpubot
Copy link
Copy Markdown
Contributor

@cpubot cpubot commented Mar 28, 2026

Implements Writer for std::io::Cursor across

  • &mut [u8]
  • [u8; N]
  • Box<[u8]>
  • Vec<u8>
  • &mut Vec<u8>

Which is the exact set of types implemented by std.

@cpubot cpubot force-pushed the std-io-cursor-write branch from e99cbe8 to 20332b8 Compare March 28, 2026 20:23
@cpubot cpubot requested a review from kskalski March 28, 2026 20:28
}

#[inline]
fn cursor_advance_slice<T>(
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.

it's only used in cursor_slice_as_trusted_for, I would just inject the impl there

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Updated

@cpubot cpubot force-pushed the std-io-cursor-write branch from 20332b8 to aed46ff Compare March 29, 2026 17:08
@cpubot cpubot force-pushed the std-io-cursor-write branch from aed46ff to 302b9c8 Compare March 29, 2026 17:19
@cpubot cpubot merged commit efae702 into master Mar 30, 2026
4 checks passed
@cpubot cpubot deleted the std-io-cursor-write branch March 30, 2026 00:34
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