-
Notifications
You must be signed in to change notification settings - Fork 12.7k
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
Rename 'cont' to 'next' #2229
Comments
Bikeshed: how about "skip"? It seems like variables are more likely to be named "next" than "skip". |
Is there anything wrong with continue? |
@Jonanin But seriously, that's the reason. |
... alright ... interesting, I guess :] |
A good rule is justified by it's exceptions. 'Next' is a very common variable name in linked lists and other data structures, I find 'skip' to be frequently used for boolean flags, while 'continue' is rarely used. That may be quite subjective but I'd rather have rust adopt 'continue'. As a compromise, I'd vote for 'resume' (which is a keyword in BASIC). |
I like |
Repasting from my mis-comment in 2233: Also keep in mind how the keyword will sound with the labeled break/continue from #2216. For a loop labeled I submit |
We talked about this on IRC yesterday and stumbled into the pleasing possibility of making |
I think overloading I do like the |
Opinions on |
I like |
I honestly think the short keywords are hurting rust. |
@mitsuhiko #554 could turn |
OK this is a trivial change to make but we don't have a clear consensus - I'm not even sure there's consensus that we want to change |
My preference lies with How about |
I like |
I like loop. Hell, you could do "recur" if you really felt like it... |
On Reddit someone suggested Perl’s "redo". |
I don't like |
We're going to go with 'again'. Did some surveying and it's the least-objectionable one that fits in the length rule. |
|
Add cache location on windows to documentation
'cont' sounds offensive, especially when spoken aloud. There's precedent for 'next' in other languages.
Remember to change the docs.
The text was updated successfully, but these errors were encountered: