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.

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

Make vec::view a method too. #3526

Merged
merged 1 commit into from
Sep 27, 2012
Merged

Make vec::view a method too. #3526

merged 1 commit into from
Sep 27, 2012

Conversation

Dretch
Copy link
Contributor

@Dretch Dretch commented Sep 18, 2012

I was surprised to find that there is no method equivalent of vec::view.

There may be a reason for this, but if not, please accept this.

@graydon
Copy link
Contributor

graydon commented Sep 19, 2012

view is vestigial / redundant, imo; all the substring-returning methods should return slices.

@brson
Copy link
Contributor

brson commented Sep 27, 2012

I want to merge this because I don't think we will get around to changing slice for 0.4

brson added a commit that referenced this pull request Sep 27, 2012
Make vec::view a method too.
@brson brson merged commit 2340ef9 into rust-lang:incoming Sep 27, 2012
@brson
Copy link
Contributor

brson commented Sep 27, 2012

Thanks!

bors pushed a commit to rust-lang-ci/rust that referenced this pull request May 15, 2021
RalfJung pushed a commit to RalfJung/rust that referenced this pull request May 19, 2024
Adjust Allocation Bytes used by Miri to custom MiriAllocBytes

Previously, the `MiriMachine` used `type Bytes = Box<[u8]>` for its allocations.
This PR swaps this out for a custom `MiriAllocBytes` type implemented in `alloc_bytes.rs`.
This is in anticipation of an extension to Miri's FFI, which will require its allocations to take care of alignment (the methods in `impl AllocBytes for Box<[u8]>` ignore this `_align: Align` argument).

Needs rust-lang#124492
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.

None yet

3 participants