Skip to content

update for rust 0.12.0-pre (or 0.11) #6

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

nathantypanski
Copy link

println was changed in Rust a while back to the println!() macro, which is now reflected in the code.

All Rust code, excluding your chain_of_responsibility.rs (unimplemented) now compiles on Rust

rustc 0.12.0-pre

commit

e178ebf681d532c1c965883ae34788713f748960

The iPhone -> IPhone change might be controversial - it's now linted by the Rust compiler (so the old code throws a warning), and while these can be hidden, the current naming conventions guidelines suggest camelcase structs ...

Also, this should compile with rustc 0.11 just as easily as with the nightlies. It's not exactly a major change.

println was changed to the println!() macro, which is now reflected in
the code.

All Rust code, excluding chain_of_responsibility.rs (unimplemented) now
compiles on Rust

    rustc 0.12.0-pre

commit

    e178ebf681d532c1c965883ae34788713f748960
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.

1 participant