Skip to content
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

Add some notes on repetition #424

Merged
merged 2 commits into from
Sep 27, 2018
Merged

Conversation

mark-i-m
Copy link
Member

cc rust-lang/rust#51934

The new Kleene op is not stable yet, but the removal of ? as a separator is. I can split this up into two PRs if you want.

@@ -59,6 +59,10 @@ matches, in a structure that mimics the structure of the repetition encountered
on a successful match. The job of the transcriber is to sort that structure
out.

> **Edition Differences**: In the 2018 edition, a new Kleene operator was
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What ? means should be defined outside of the edition differences section. And then the edition differences section should just state that it's not available in the 2015 edition.

@Havvy Havvy added New Content Missing features or aspects of language not currently documented. RFC Stabilization Docs Documentation required for stabilizing a feature labels Sep 25, 2018
@mark-i-m
Copy link
Member Author

@Havvy better now?

@Havvy Havvy merged commit e09454d into rust-lang:master Sep 27, 2018
@Havvy
Copy link
Contributor

Havvy commented Sep 27, 2018

💟 Thanks!

Looking at the tracking issue, I think this is already stable in nightly Rust. If it's not, we can revert the two commits.

@mark-i-m
Copy link
Member Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
New Content Missing features or aspects of language not currently documented. RFC Stabilization Docs Documentation required for stabilizing a feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants