Skip to content

libcore/num: Remove unused macro argument. #31663

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 1 commit into from
Feb 15, 2016

Conversation

nodakai
Copy link
Contributor

@nodakai nodakai commented Feb 14, 2016

No description provided.

It is no longer necessary after dd0d495

Signed-off-by: NODA, Kai <nodakai@gmail.com>
@rust-highfive
Copy link
Contributor

r? @brson

(rust_highfive has picked a reviewer for you, use r? to override)

@hanna-kruppe
Copy link
Contributor

This is not the first macro to end up with an unused argument. It's not even the first one in libcore/num! Makes me wonder if an unused arguments lint for macros is a good idea (if it's even possible).

@alexcrichton
Copy link
Member

@bors: r+ e03cea4

Thanks!

@bors
Copy link
Collaborator

bors commented Feb 15, 2016

⌛ Testing commit e03cea4 with merge 7a737d5...

@bors bors merged commit e03cea4 into rust-lang:master Feb 15, 2016
@nodakai nodakai deleted the cleanup-uint_module branch March 8, 2016 05:03
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants