Skip to content
This repository has been archived by the owner on Jan 25, 2022. It is now read-only.

Mark do as a keyword and do catch as a control keyword. #102

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

Conversation

KalitaAlexey
Copy link
Collaborator

The feature do catch has been published to Rust's repository.

Fixes #100

@KalitaAlexey
Copy link
Collaborator Author

@cramertj,
Will do catch be stabilized?

@cramertj
Copy link

No-- do catch is (most likely) just a stand in while we work out the stabilization process for catch { ... }.

@KalitaAlexey
Copy link
Collaborator Author

@cramertj,
How long will it be with us?

@cramertj
Copy link

It's hard to tell exactly, but changing it is blocked on removing catch as a valid struct name (see here).

I'd imagine it's at least a version or two away, since I think the current consensus is that such a change would need at least a warning cycle.

@KalitaAlexey
Copy link
Collaborator Author

Okay.
At least six weeks is a long time.
I am going to merge it and when it changes, I will open another PR.

@zargony,
What do you think?

@zargony
Copy link
Owner

zargony commented Jun 27, 2017

Darn, I didn't get any notification of your ping again. Github doesn't like me anymore. Sorry for not catching this :-(

I didn't follow do catch development in Rust. How did it turn out?

@KalitaAlexey
Copy link
Collaborator Author

@zargony,
Nothing interesting so far.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants