The default bug view has changed. See this FAQ.

Status

()

Core
JavaScript Engine
--
enhancement
RESOLVED FIXED
9 years ago
9 years ago

People

(Reporter: Jesse Ruderman, Assigned: Jesse Ruderman)

Tracking

({fixed1.8.1.15})

Trunk
x86
Mac OS X
fixed1.8.1.15
Points:
---
Bug Flags:
in-testsuite -
in-litmus -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Assignee)

Description

9 years ago
Created attachment 309554 [details] [diff] [review]
patch

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+

Updated

9 years ago
Assignee: general → jruderman
(Assignee)

Updated

9 years ago
Status: NEW → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED

Updated

9 years ago
Flags: in-testsuite-
Flags: in-litmus-
Created attachment 316773 [details] [diff] [review]
patch for MOZILLA_1_8_BRANCH

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+
Attachment #316773 - Flags: approval1.8.1.15?
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

9 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.