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

Issue 7655: align the bench printouts so numbers tend to be aligned #9823

Merged
merged 2 commits into from
Oct 14, 2013

Conversation

pnkfelix
Copy link
Member

r? anyone.

You can see a bit more discussion on #7655.

This does not close any ticket; I am just scratching an itch. Note in particular that I picked the value {:>9} ns/iter pretty much out of a hat. :)

…aligned.

(scratching an itch.)

Rebased and updated.
Fixed bug: omitted field init from embedded struct literal in a test.
Fixed bug: int underflow during padding length calculation.
@pnkfelix
Copy link
Member Author

r=alexcrichton

alex also pointed out:

I think right now our maximum is 1s, so if you take more than 10^9ns you may already have problems :)

bors added a commit that referenced this pull request Oct 14, 2013
…crichton

r? anyone.

You can see a bit more discussion on #7655.

This does not close any ticket; I am just scratching an itch.  Note in particular that I picked the value `{:>9} ns/iter` pretty much out of a hat.  :)
@bors bors closed this Oct 14, 2013
@bors bors merged commit 602b3cd into rust-lang:master Oct 14, 2013
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.

3 participants