Skip to content

Conversation

@aochagavia
Copy link
Contributor

No description provided.

@alexcrichton
Copy link
Member

cc @bjz

Copy link
Contributor

Choose a reason for hiding this comment

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

this last line isn't terminated.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks! It should be fixed now.

@brendanzab
Copy link
Contributor

Looks fine to me.

Copy link
Member

Choose a reason for hiding this comment

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

These may also be expressed as self.div_rem(other).val0().

@alexcrichton
Copy link
Member

Could you squash the last commit into one of the first two? Other than that I think this is good to go!

@aochagavia
Copy link
Contributor Author

Done!

@bors bors closed this Jun 10, 2014
@bors bors merged commit 0eb858b into rust-lang:master Jun 10, 2014
bors added a commit that referenced this pull request Jun 10, 2014
Closes #14611 (std: Remove the as_utf16_p functions)
Closes #14694 (Num cleanup)
Closes #14760 (Add --color to test binary options)
Closes #14763 (std: Move dynamic_lib from std::unstable to std)
Closes #14766 (Add test for issue #13446)
Closes #14769 (collections: Add missing Default impls)
Closes #14773 (General nits)
Closes #14776 (rustdoc: Correctly classify enums/typedefs)
@aochagavia aochagavia deleted the num-cleanup branch June 15, 2014 10:07
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.

5 participants