Closed
Bug 423042
Opened 16 years ago
Closed 16 years ago
js.c Print should flush
Categories
(Core :: JavaScript Engine, enhancement)
Tracking
()
RESOLVED
FIXED
People
(Reporter: jruderman, Assigned: jruderman)
Details
(Keywords: fixed1.8.1.15)
Attachments
(2 files)
813 bytes,
patch
|
shaver
:
review+
shaver
:
approval1.9+
|
Details | Diff | Splinter Review |
820 bytes,
patch
|
shaver
:
review+
dveditz
:
approval1.8.1.15+
|
Details | Diff | Splinter Review |
This patch prevents jsfunfuzz logs from being cut off when ./js crashes. So far, I've been recommending to people who use jsfunfuzz that they should apply this patch. Is the performance hit is small enough that it's ok to include it in js.c instead?
Comment on attachment 309554 [details] [diff] [review] patch Performance of js.c's Print isn't important, r+a=shaver for NPOTB jesse-helping patch.
Attachment #309554 -
Flags: review+
Attachment #309554 -
Flags: approval1.9+
Assignee | ||
Updated•16 years ago
|
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Updated•16 years ago
|
Flags: in-testsuite-
Flags: in-litmus-
Comment 2•16 years ago
|
||
This is the corresponding patch for MOZILLA_1_8_BRANCH.
Attachment #316773 -
Flags: review?(shaver)
Comment on attachment 316773 [details] [diff] [review] patch for MOZILLA_1_8_BRANCH r=shaver
Attachment #316773 -
Flags: review?(shaver) → review+
Updated•16 years ago
|
Attachment #316773 -
Flags: approval1.8.1.15?
Comment 4•16 years ago
|
||
Comment on attachment 316773 [details] [diff] [review] patch for MOZILLA_1_8_BRANCH Approved for 1.8.1.15, a=dveditz for release-drivers
Attachment #316773 -
Flags: approval1.8.1.15? → approval1.8.1.15+
Assignee | ||
Comment 5•16 years ago
|
||
I checked in the branch patch. Thanks for porting it, Gary!
Keywords: fixed1.8.1.15
You need to log in
before you can comment on or make changes to this bug.
Description
•