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

Hygiene for 3 kinds of args #15425

Merged
merged 13 commits into from
Jul 5, 2014

Conversation

jbclements
Copy link
Contributor

This pull request adds hygiene for 3 kinds of argument bindings:

  • arguments to item fns,
  • arguments to ExprFnBlocks, and
  • arguments to ExprProcs

It also adds a bunch of unit tests, fixes a few macro uses to be non-capturing, and has a few cleanup items.

local make check succeeds.

@emberian
Copy link
Member

emberian commented Jul 4, 2014

r=me with nit fixed

@jbclements
Copy link
Contributor Author

Okay, nit fixed in aa91ef0 .

bors added a commit that referenced this pull request Jul 5, 2014
…=cmr

This pull request adds hygiene for 3 kinds of argument bindings:
- arguments to item fns,
- arguments to `ExprFnBlock`s, and
- arguments to `ExprProc`s

It also adds a bunch of unit tests, fixes a few macro uses to be non-capturing, and has a few cleanup items.

local `make check` succeeds.
@bors bors closed this Jul 5, 2014
@bors bors merged commit cc13f9b into rust-lang:master Jul 5, 2014
@huonw
Copy link
Member

huonw commented Jul 5, 2014

Does this close #9383?

@jbclements jbclements mentioned this pull request Jul 9, 2014
bors added a commit to rust-lang-ci/rust that referenced this pull request Nov 13, 2023
…ock, r=Veykril

minor : Deunwrap convert_comment_block and desugar_doc_comment

Closes subtask 13 of rust-lang#15398 . I still don't know a more idiomatic way for the for loops I added, any suggestion would make me happy.
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.

4 participants