Skip to content

Conversation

@steveklabnik
Copy link
Contributor

This will hopefully help people with their first steps in Rust.

Fixes #16143.

/cc @jvns

src/doc/guide.md Outdated
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks, fixing

This will hopefully help people with their first steps in Rust.

Fixes rust-lang#16143.
bors referenced this pull request Dec 16, 2014
This creates an enormous amount of spew.
bors added a commit that referenced this pull request Dec 16, 2014
Add comments with type annotations.

Reviewed-by: alexcrichton
bors referenced this pull request Dec 16, 2014
Using a type alias for iterator implementations is fragile since this
exposes the implementation to users of the iterator, and any changes
could break existing code.

This commit changes the iterators of `VecMap` to use
proper new types, rather than type aliases.  However, since it is
fair-game to treat a type-alias as the aliased type, this is a:

[breaking-change].
bors added a commit that referenced this pull request Dec 16, 2014
Add comments with type annotations.

Reviewed-by: alexcrichton
brson added a commit to brson/rust that referenced this pull request Dec 16, 2014
This will hopefully help people with their first steps in Rust.

Fixes rust-lang#16143.

/cc @jvns
bors added a commit that referenced this pull request Dec 17, 2014
Add comments with type annotations.

Reviewed-by: alexcrichton
alexcrichton added a commit to alexcrichton/rust that referenced this pull request Dec 17, 2014
This will hopefully help people with their first steps in Rust.

Fixes rust-lang#16143.

/cc @jvns
@bors bors merged commit 31b240d into rust-lang:master Dec 17, 2014
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.

Consider adding type annotations in comments

5 participants