Skip to content

Issue in Rust docs for Ty in libsyntax #22721

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

Closed
jroesch opened this issue Feb 23, 2015 · 3 comments
Closed

Issue in Rust docs for Ty in libsyntax #22721

jroesch opened this issue Feb 23, 2015 · 3 comments

Comments

@jroesch
Copy link
Member

jroesch commented Feb 23, 2015

This isn't a big deal but the documentation is slightly out of date for the TyFixedLengthVec variant of ast::Ty.

Reference: http://doc.rust-lang.org/nightly/src/syntax/ast.rs.html#1247.

The comment should read:

/// A fixed length array (`[T; n]`)

I will fix this if I have time, otherwise it is a small tweak that someone looking to contribute could make.

cc @steveklabnik

@jroesch jroesch changed the title Small issue in Rust docs for libsyntax Issue in Rust docs for Ty in libsyntax Feb 23, 2015
@munds
Copy link
Contributor

munds commented Feb 23, 2015

I've taken the liberty of fixing that comment :)

https://github.com/mund/rust/commit/d9fe33522bf12ab660803f1781a646656268e637

@steveklabnik
Copy link
Member

@mund can you open up a pull request please?

@munds
Copy link
Contributor

munds commented Feb 26, 2015

Yep, done!

#22845

Is this the right way to do it?

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

No branches or pull requests

3 participants