Skip to content

Macros: Split in topics, use playpen and cover more topics #193

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

Closed
wants to merge 2 commits into from

Conversation

japaric
Copy link
Member

@japaric japaric commented Jul 22, 2014

I've rebased #185 and done a few changes. I'm satisfied with the current examples and the contents, but the wording could be further improved.

We are still missing the topic on how to export macros to be able to close #110

@mdinger I hope you can review the changes

@mdinger
Copy link
Contributor

mdinger commented Jul 22, 2014

I'll look at it but I may not get to it for a day or 2. Not sure.

@japaric
Copy link
Member Author

japaric commented Jul 22, 2014

I'll look at it but I may not get to it for a day or 2. Not sure.

Thanks in advance!

* `path`
* `stmt`
* `tt` (*token tree*) is used for operators (`+`, `-`, etc)
* `ty` (*type*)
Copy link
Contributor

Choose a reason for hiding this comment

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

It does make sense to only explain the common ones but having most of them blank here looks really weird. A brief description might be in order. Also, that Macro by example page still doesn't describe them so linking to it won't be of much help.

@mdinger mdinger mentioned this pull request Jul 24, 2014

* `block`
* `expr` is used for expressions
* `ident` is used for variable/function names

Choose a reason for hiding this comment

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

I think it's worth adding (*identifier*) here. Same with for expr, pat and stmt. Note: I actually have no idea what stmt is.

Copy link
Contributor

Choose a reason for hiding this comment

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

  • stmt (statement) (from IRC)
  • pat (pattern) (not sure though...asked on IRC and got no reply)
  • matchers (lhs of the => in macro rules) (from manual)
  • tt (rhs of the => in macro rules) (from manual; token tree from IRC)

@mdinger
Copy link
Contributor

mdinger commented Aug 14, 2014

This has stalled...I think you should:

  • Drop remove the git-ignore change I added mdinger@6900fbb
  • Add some of these descriptions (the ones you think fit) to examples/staging/macros/arguments/input.md:
  • stmt (statement) (from IRC)
  • pat (pattern) (not sure though...asked on IRC and got no reply)
  • matchers (lhs of the => in macro rules) (from manual)
  • tt (rhs of the => in macro rules) (from manual; token tree from IRC)

@steveklabnik
Copy link
Member

@japaric , I'm assuming you're not interested in updating this PR, is that correct?

@japaric
Copy link
Member Author

japaric commented Dec 24, 2014

I'm assuming you're not interested in updating this PR, is that correct?

Correct. I'm closing this to clear the queue.

@japaric japaric closed this Dec 24, 2014
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.

Split current "macro_rules" chapter ", cover more topics
4 participants