comprehensive-rust
comprehensive-rust copied to clipboard
Tuples with "..." make no sense, so don't use them
Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
View this failed invocation of the CLA check for more information.
For the most up to date status, view the checks section at the bottom of the pull request.
Hey @daym, thanks for the PR!
I was actually trying to use ... as a place holder there, meaning that there could be more Tn types following. But I see now that it can be understood as being part of the syntax.
How does it look if we include a few different tuple sizes in the example? Like (), (T,), (T1, T2). I think that could make it clearer without using made-up syntax.
Hi @daym, I updated the branch with my suggestion above. If you sign the CLA, then we can merge this. Thanks!
Hi,
How does it look if we include a few different tuple sizes in the example? Like
(), (T,), (T1, T2). I think that could make it clearer without using made-up syntax.
Yes, I took a look and that looks good to me!
Also, I've signed the CLA now.
Yes, I took a look and that looks good to me!
Also, I've signed the CLA now.
Thanks a lot!