Closed
Bug 521364
Opened 15 years ago
Closed 15 years ago
Make jitstats output not miss stats
Categories
(Core :: JavaScript Engine, defect)
Tracking
()
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
status1.9.2 | --- | beta4-fixed |
People
(Reporter: bzbarsky, Assigned: bzbarsky)
Details
(Whiteboard: fixed-in-tracemonkey)
Attachments
(1 file)
6.20 KB,
patch
|
luke
:
review+
|
Details | Diff | Splinter Review |
I discovered today that the timeoutIntoInterpreter stat isn't listed for some reason. Let's not have that happen again.
Assignee | ||
Comment 1•15 years ago
|
||
Attachment #405389 -
Flags: review?
Assignee | ||
Updated•15 years ago
|
Attachment #405389 -
Flags: review? → review?(dvander)
Updated•15 years ago
|
Attachment #405389 -
Flags: review?(dvander) → review+
Comment 2•15 years ago
|
||
Comment on attachment 405389 [details] [diff] [review] Like so Stolen, at bz's request. Cool patch, helps me be lazy. >+ debug_only_printf(LC_TMStats, "%s", "recorder"); >+ debug_only_printf(LC_TMStats, "%s", "\n"); >+ debug_only_printf(LC_TMStats, "%s", "monitor"); >+ debug_only_printf(LC_TMStats, "%s", "\n"); I suspect you used the eta-expanded form because debug_only_printf(LC_TMStats, "recorder") was an error. For that we have debug_only_print0. Yay macros!
Assignee | ||
Comment 3•15 years ago
|
||
> debug_only_printf(LC_TMStats, "recorder") was an error. Yep. ;) > For that we have debug_only_print0. Aha! Perfect. Will change to that.
Assignee | ||
Comment 4•15 years ago
|
||
Pushed http://hg.mozilla.org/tracemonkey/rev/9ca9a164711c with the debug_only_print0
Whiteboard: fixed-in-tracemonkey
Comment 5•15 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/9ca9a164711c
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Comment 6•15 years ago
|
||
http://hg.mozilla.org/releases/mozilla-1.9.2/rev/ce866c82a025
status1.9.2:
--- → final-fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•