1322380 - Improve GC profiling output

Status: RESOLVED FIXED

(Core :: JavaScript: GC, defect)

Comment 1

[Jon Coppeard (:jonco)]

Attachment: print-profile-headers-together

At the moment the header lines which describe what each field means are printed at different times for the nursery and major GC profile, depending on how many lines are output for each.  This makes the output even more confusing when these are both enabled.

This patch prints them both at the same time.

Comment 2

[Jon Coppeard (:jonco)]

Attachment: align-profile-headers

The major GC profile information doesn't line up correctly under the column headings because they're longer than six characters.  This patch fixes that by shortening the column names.

Comment 3

[Steve Fink [:sfink] [:s:]]

Comment on attachment 8817210 [details] [diff] [review]
print-profile-headers-together

Review of attachment 8817210 [details] [diff] [review]:
-----------------------------------------------------------------

nice cleanup

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/d01ac7a95ee4
Print major / minor GC profiling headers together if both enabled r=sfink
https://hg.mozilla.org/integration/mozilla-inbound/rev/c2626a3d815c
Fix alignment of major GC profiling headers r=sfink

Comment 5

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/d01ac7a95ee4
https://hg.mozilla.org/mozilla-central/rev/c2626a3d815c