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

Allow trailing commas in bitflags! macro #16986

Merged
merged 4 commits into from Sep 4, 2014

Conversation

Projects
None yet
4 participants
@brendanzab
Copy link
Member

brendanzab commented Sep 4, 2014

Closes #16469

@alexcrichton

This comment has been minimized.

Copy link
Member

alexcrichton commented Sep 4, 2014

Looks good to me! r=me with a rebase

brendanzab added some commits Sep 4, 2014

Attach documentation to the bitflags! macro itself
This is in consistent with the style demonstrated in the std::macros module.
Use {} for bitflags! definition and invocations
This looks nicer because it reflects Rust's other syntactic structures.

@brendanzab brendanzab force-pushed the brendanzab:bitflags branch from 32bf88f to ef354d8 Sep 4, 2014

@brendanzab

This comment has been minimized.

Copy link
Owner Author

brendanzab commented on ef354d8 Sep 4, 2014

r=alexcrichton

@bors

This comment has been minimized.

Copy link
Contributor

bors commented on ef354d8 Sep 4, 2014

saw approval from alexcrichton
at brendanzab@ef354d8

This comment has been minimized.

Copy link
Contributor

bors replied Sep 4, 2014

merging bjz/rust/bitflags = ef354d8 into auto

This comment has been minimized.

Copy link
Contributor

bors replied Sep 4, 2014

bjz/rust/bitflags = ef354d8 merged ok, testing candidate = e024017

This comment has been minimized.

Copy link
Contributor

bors replied Sep 4, 2014

fast-forwarding master to auto = e024017

bors added a commit that referenced this pull request Sep 4, 2014

@bors bors closed this Sep 4, 2014

@bors bors merged commit ef354d8 into rust-lang:master Sep 4, 2014

1 of 2 checks passed

continuous-integration/travis-ci The Travis CI build failed
Details
default all tests passed

@brendanzab brendanzab deleted the brendanzab:bitflags branch Sep 4, 2014

@theemathas

This comment has been minimized.

Copy link

theemathas commented on ca60b74 Jan 16, 2015

Seriously, what exactly is this commit?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.