Skip to content

Feature gate all quasi-quoting macros. #12025

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

Merged
merged 2 commits into from
Feb 5, 2014
Merged

Conversation

lilac
Copy link
Contributor

@lilac lilac commented Feb 4, 2014

Closes #11630.

@@ -49,6 +49,12 @@ static KNOWN_FEATURES: &'static [(&'static str, Status)] = &[
("trace_macros", Active),
("simd", Active),
("default_type_params", Active),
("quote_tokens", Active),
Copy link
Member

Choose a reason for hiding this comment

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

Maybe these could all be covered by the same gate (e.g. called quote)?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, I didn't think about it. It seems that there was only one "quote" macro, but later split to multiple ones.
I am not sure if there is a gain to keep them separate in feature gate as well.
If not, I am happy to replace with a single gate, that is simpler.

Copy link
Member

Choose a reason for hiding this comment

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

Yes to the single, simple quote feature gate.

@brson
Copy link
Contributor

brson commented Feb 4, 2014

Thanks!

bors added a commit that referenced this pull request Feb 5, 2014
@bors bors closed this Feb 5, 2014
@bors bors merged commit 124938b into rust-lang:master Feb 5, 2014
@lilac lilac deleted the feature-gate-quote branch February 5, 2014 10:49
bors added a commit to rust-lang-ci/rust that referenced this pull request Jul 25, 2022
minor: Document completion context some more
flip1995 pushed a commit to flip1995/rust that referenced this pull request Jan 11, 2024
Include GitLab in the CI section of the clippy doc book

Fixes rust-lang#12012
changelog: Docs: [`Continuous Integration`] now includes how to use clippy in GitLab CI.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feature gate all quasi-quoting macros
4 participants