Skip to content

Syntax: Rewrite macro_rules - #326

Merged
bors[bot] merged 1 commit into
rust-lang:masterfrom
ehuss:syntax-macro-rules
Aug 20, 2018
Merged

bors[bot] merged 1 commit into
rust-lang:masterfrom
ehuss:syntax-macro-rules

Conversation

@ehuss

@ehuss ehuss commented Aug 6, 2018

Copy link
Copy Markdown

macro_rules failed in a lot of cases, this reworks it to fully support:

  • Complete matcher syntax.
  • Better bracket balancing support.
  • Add lifetime designator (1.28).
  • More resilient to odd syntax within the transcriber. The content doesn't have
    to be Rust syntax, so it can sometimes get confused. Add a few escapes so it
    at least doesn't barf.
  • At-most-once kleene operator (2018).

Closes #311

image

@ehuss

ehuss commented Aug 20, 2018

Copy link
Copy Markdown
Author

bors r+

@bors

bors Bot commented Aug 20, 2018

Copy link
Copy Markdown

Merge conflict (retrying...)

7 similar comments
@bors

bors Bot commented Aug 20, 2018

Copy link
Copy Markdown

Merge conflict (retrying...)

@bors

bors Bot commented Aug 20, 2018

Copy link
Copy Markdown

Merge conflict (retrying...)

@bors

bors Bot commented Aug 20, 2018

Copy link
Copy Markdown

Merge conflict (retrying...)

@bors

bors Bot commented Aug 20, 2018

Copy link
Copy Markdown

Merge conflict (retrying...)

@bors

bors Bot commented Aug 20, 2018

Copy link
Copy Markdown

Merge conflict (retrying...)

@bors

bors Bot commented Aug 20, 2018

Copy link
Copy Markdown

Merge conflict (retrying...)

@bors

bors Bot commented Aug 20, 2018

Copy link
Copy Markdown

Merge conflict (retrying...)

@bors

bors Bot commented Aug 20, 2018

Copy link
Copy Markdown

Merge conflict

macro_rules failed in a lot of cases, this reworks it to fully support:
- Complete matcher syntax.
- Better bracket balancing support.
- Add `lifetime` designator (1.28).
- More resilient to odd syntax within the transcriber.  The content doesn't have
  to be Rust syntax, so it can sometimes get confused. Add a few escapes so it
  at least doesn't barf.
- At-most-once kleene operator (2018).
@ehuss
ehuss force-pushed the syntax-macro-rules branch from 32dd27c to b9a965f Compare August 20, 2018 22:15
@ehuss

ehuss commented Aug 20, 2018

Copy link
Copy Markdown
Author

bors r+

@bors

bors Bot commented Aug 20, 2018

Copy link
Copy Markdown

Merge conflict (retrying...)

bors Bot added a commit that referenced this pull request Aug 20, 2018
326: Syntax: Rewrite macro_rules r=ehuss a=ehuss

macro_rules failed in a lot of cases, this reworks it to fully support:
- Complete matcher syntax.
- Better bracket balancing support.
- Add `lifetime` designator (1.28).
- More resilient to odd syntax within the transcriber.  The content doesn't have
  to be Rust syntax, so it can sometimes get confused. Add a few escapes so it
  at least doesn't barf.
- At-most-once kleene operator (2018).

Closes #311 

<img width="903" alt="image" src="https://user-images.githubusercontent.com/43198/43736311-018cc660-9972-11e8-8931-d4d2bd5e7501.png">


Co-authored-by: Eric Huss <eric@huss.org>
@bors

bors Bot commented Aug 20, 2018

Copy link
Copy Markdown

Build succeeded

@bors
bors Bot merged commit b9a965f into rust-lang:master Aug 20, 2018
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.

1 participant