Skip to content

the book should mention the difference between vectors and arrays. #20948

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
Closed
bharadwaj6 opened this issue Jan 11, 2015 · 3 comments · Fixed by #22305
Closed

the book should mention the difference between vectors and arrays. #20948

bharadwaj6 opened this issue Jan 11, 2015 · 3 comments · Fixed by #22305

Comments

@bharadwaj6
Copy link

I have found out that vectors are heap allocated whereas arrays are stack allocated, but this was not mentioned anywhere in the book.
@kmcallister
Copy link
Contributor

Arrays are allocated directly within the containing structure. So Box<[u8; 3]> is heap allocated, but let x: [u8; 3] = ... is stack-allocated like any let variable.

@bharadwaj6
Copy link
Author

Let me summarize it here: So unless we box the array declaration, arrays would be allocated on the stack, whereas vectors would always be allocated on the heap (possibly because they are implemented as box types in vec!?). Correct me if I am wrong here.

@steveklabnik steveklabnik changed the title rustbook: should mention the difference between vectors and arrays. the book should mention the difference between vectors and arrays. Jan 11, 2015
@steveklabnik
Copy link
Member

Yes.

@bharadwaj6 bharadwaj6 mentioned this issue Jan 12, 2015
Closed
@steveklabnik steveklabnik mentioned this issue Feb 13, 2015
Merged
steveklabnik added a commit to steveklabnik/rust that referenced this issue Feb 13, 2015
@steveklabnik
Note that Vec<T> is heap allocated.
6d2e3d4 
Fixes rust-lang#20948
steveklabnik added a commit to steveklabnik/rust that referenced this issue Feb 14, 2015
@steveklabnik
Rollup merge of rust-lang#22305 - steveklabnik:gh20948, r=alexcrichton
6d3cb62 
Fixes rust-lang#20948
steveklabnik added a commit to steveklabnik/rust that referenced this issue Feb 15, 2015
@steveklabnik
Rollup merge of rust-lang#22305 - steveklabnik:gh20948, r=alexcrichton
4587f64 
Fixes rust-lang#20948
Manishearth added a commit to Manishearth/rust that referenced this issue Feb 15, 2015
@Manishearth
Rollup merge of rust-lang#22305 - steveklabnik:gh20948, r=alexcrichton
70647ec 
 Fixes rust-lang#20948
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Note that Vec<T> is heap allocated. steveklabnik/rust
3 participants
@steveklabnik @kmcallister @bharadwaj6