Skip to content

Adjust ljust#16

Merged
hsbt merged 1 commit into
ruby:masterfrom
kaiquekandykoga:length-ljust
Nov 8, 2024
Merged

Adjust ljust#16
hsbt merged 1 commit into
ruby:masterfrom
kaiquekandykoga:length-ljust

Conversation

@kaiquekandykoga

@kaiquekandykoga kaiquekandykoga commented Aug 8, 2022

Copy link
Copy Markdown
Contributor

Benchmark#bm with labels was not using the highest length
among the labels to adjust the correct ljust. Instead of
printing the result during the report generation,
now it is waiting to print the result once it is generated.

Benchmark.bm { |x|
  x.item("aaaa") { 1 }
  x.item("aaaaaaaa") { 0 }
}

After

              user     system      total        real
aaaa      0.000005   0.000002   0.000007 (  0.000003)
aaaaaaaa  0.000001   0.000001   0.000002 (  0.000002)

Before

       user     system      total        real
aaaa  0.000005   0.000001   0.000006 (  0.000003)
aaaaaaaa  0.000002   0.000001   0.000003 (  0.000003)

Benchmark#bm with labels was not using the highest length
among the labels to adjust the correct ljust. Instead of
printing the result during the report generation,
now it is waiting to print the result once it is generated.

Benchmark.bm { |x|
  x.item("aaaa") { 1 }
  x.item("aaaaaaaa") { 0 }
}

After

              user     system      total        real
aaaa      0.000005   0.000002   0.000007 (  0.000003)
aaaaaaaa  0.000001   0.000001   0.000002 (  0.000002)

Before

       user     system      total        real
aaaa  0.000005   0.000001   0.000006 (  0.000003)
aaaaaaaa  0.000002   0.000001   0.000003 (  0.000003)
@hsbt
hsbt merged commit b768760 into ruby:master Nov 8, 2024
@hsbt

hsbt commented Nov 8, 2024

Copy link
Copy Markdown
Member

👍

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.

2 participants