Skip to content

Doc: corrects obsolete pointer syntax #18568

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

Merged
merged 1 commit into from
Nov 4, 2014
Merged

Conversation

gamazeps
Copy link
Contributor

@gamazeps gamazeps commented Nov 3, 2014

Goes from ~ to box
Closes #18551

@steveklabnik
Copy link
Member

Thank you!!!

@gereeter
Copy link
Contributor

gereeter commented Nov 3, 2014

Shouldn't it be vec![], not box []?

@gamazeps
Copy link
Contributor Author

gamazeps commented Nov 3, 2014

@gereeter Quite honestly, I just updated the content of the doc to the new syntax, but yes, it seems that the old doc was a bit weird.
If you're sure it's should be vec![], I'll gladly change it (as I'm not really confident in what I changed (let's say that I'm sure that at least it's not wronger than it used to be))

alexcrichton added a commit to alexcrichton/rust that referenced this pull request Nov 4, 2014
@brendanzab
Copy link
Member

I can confirm that it should be vec![]. The variant is defined as: PatVec(Vec<P<Pat>>, Option<P<Pat>>, Vec<P<Pat>>),

@steveklabnik
Copy link
Member

Yes, technically Box<[T, ..N]> is a type, but is less useful than Vec<T>.

@bors bors merged commit 9bc4da3 into rust-lang:master Nov 4, 2014
@gamazeps gamazeps deleted the issue18551 branch May 7, 2017 14:52
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.

Obsolete pointer syntax in nightly documentation
5 participants