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

Make ast::Arg usable in quote macros #14722

Merged
merged 1 commit into from
Jun 7, 2014
Merged

Make ast::Arg usable in quote macros #14722

merged 1 commit into from
Jun 7, 2014

Conversation

hannobraun
Copy link

The need for this came up while I worked on a procedural macro that parses an argument and adds the same argument to a generated function.

This makes ast::Arg usable in the quote_ macros.

Please note that this commit doesn't include a regression test. There
are tests that use the quote macros, but all of them are ignored. Due to
that, there is no obvious (to me) way to test this.

Since this change is absolutely trivial and only hooks up an additional
type to existing infrastructure (which presumably is tested elsewhere),
I concluded it's not worth the effort to follow up on this.
hannobraun added a commit to hannobraun/vndf-2016 that referenced this pull request Jun 7, 2014
This adds the ability use the quote_ macros for ast::Arg, which is
required by librustecs.

Here's the relevant pull request:
rust-lang/rust#14722
bors added a commit that referenced this pull request Jun 7, 2014
The need for this came up while I worked on a procedural macro that parses an argument and adds the same argument to a generated function.
@bors bors closed this Jun 7, 2014
@bors bors merged commit cc28ae5 into rust-lang:master Jun 7, 2014
@hannobraun hannobraun deleted the quote-args branch June 8, 2014 19:20
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.

None yet

2 participants