Implement ReadBytesAt for owned CachedReadBytes

Question

Implement ReadBytesAt for owned CachedReadBytes

pjht opened this issue 2 years ago · 3 comments

Right now, ReadBytesAt is only implemented for a mutable reference to a CachedReadBytes. This requires an extra variable to hold the CachedReadBytes instead of passing it directly to File::open_stream. Ideally, ReadBytesAt would also be implemented for an owned CachedReadBytes, allowing it to be passed directly.

Answer 1 · 2022-10-23T22:24:12.000Z

Thanks for the suggestion, what do you think of #22?

Answer 2 · 2022-10-23T22:43:56.000Z

Looks good, that would accomplish what I wanted.

Answer 3 · 2022-10-24T21:48:37.000Z

Cool. I bundled this change into crate v0.4.0.