Skip to content

Conversation

onkis
Copy link

@onkis onkis commented Nov 30, 2014

Just a small thing I noticed while reading the tutorial.

small typo
@rust-highfive
Copy link
Contributor

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @nikomatsakis (or someone else) soon.

@sfackler
Copy link
Member

The compiler can determine that the 10 is an int through type inference so the explicit suffix isn't necessary. The prevailing style is to avoid suffixes except when the compiler can't figure the type out itself.

@alexcrichton
Copy link
Member

I agree with @sfackler in that the suffix here is not necessary and the current style is to avoid suffixes as much as possible. Thanks for the PR though @onkis!

@onkis
Copy link
Author

onkis commented Dec 1, 2014

Awesome. Thanks for getting back to this quickly

On Nov 30, 2014, at 3:05 AM, Alex Crichton notifications@github.com wrote:

Closed #19412 #19412.


Reply to this email directly or view it on GitHub #19412 (comment).

lnicola pushed a commit to lnicola/rust that referenced this pull request Apr 28, 2025
chore: Remove some unnecessary usage of `Semantics`
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.

5 participants