Uh oh!
There was an error while loading. Please reload this page.
Book - short explanation of arity - #22368
Conversation
rust-highfive
commented
Feb 15, 2015
Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @steveklabnik (or someone else) soon. If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. The way Github handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes. Please see CONTRIBUTING.md for more information. |
steveklabnik
commented
Feb 15, 2015
I have additional plans here, but in the meantime, this is fine. Thank you! |
steveklabnik
commented
Feb 15, 2015
@bors: r+ 89822el rollup |
bors
commented
Feb 15, 2015
🙀 You have the wrong number! Please try again with |
steveklabnik
commented
Feb 15, 2015
thiagooak
commented
Feb 15, 2015
@steveklabnik if you can share what you plan, or a issue number I may be able to contribute :) |
steveklabnik
commented
Feb 15, 2015
I want to make a "glossary" chapter that this and some other things (like 'parametric polymorphism') can just link to |
thiagooak
commented
Feb 15, 2015
I like the idea, will try it out |
There was a problem hiding this comment.
Why introduce the concept of "arity" here at all? "length" should be good enough, and I've never seen the word "arity" being used in this context in discussions about Rust.
There was a problem hiding this comment.
On #21568 there is a big discussion on this topic.
As for this commit, I think it is kind of a middle ground. And a concrete improvement.
But I guess length would also work
mdinger
commented
Feb 16, 2015
@steveklabnik What made you change your mind? In rust-lang/rust-by-example#97 you decided we don't need glossaries. They're too difficult. Now you want one? |
steveklabnik
commented
Feb 16, 2015
via email
There's been a few things, but I think it's mostly that that's Rust by
Example, and this is the book. I'm also still not 100% sure this is the
best fix, giving it a try and seeing how it goes. |
mdinger
commented
Feb 16, 2015
I think it's a good idea. Definitely try it before rejecting it. Can be more direct and pertinent to Rust than a wikipedia article could be. |
fix: Fully support pattern types
#21568