Closed
Bug 3649
Opened 26 years ago
Closed 21 years ago
js.c lacks gc-checking branch callback.
Categories
(Core :: JavaScript Engine, enhancement, P5)
Core
JavaScript Engine
Tracking
()
RESOLVED
FIXED
Future
People
(Reporter: mike+mozilla, Assigned: beard)
References
Details
Attachments
(1 file)
(deleted),
text/plain
|
Details |
I've copied text from bug 2235 (marked as a dup of this bug) which discusses
this problem - it turned up with a user that was using the java-enabled
javascript standalone shell and noticed very bad memory behavior. My
explanation is at the bottom... roger, could you confirm my understanding of the
problem?
Mozilla presumably does register a branch callback, so this shouldn't be a
problem with the browser.
Reporter | ||
Updated•26 years ago
|
Status: NEW → ASSIGNED
Priority: P3 → P5
Reporter | ||
Comment 2•26 years ago
|
||
Copied text from 2235 (marked as a dup of this one)
This memory blowup turned out to be a flaw with the js.c standalone interpreter
(or possibly jsj, the liveconnect-enabled interpreter.) Most applications using
js, such as the browser, register a 'branch callback' function that gets called
whenever the javascript interpreter makes a branch. Usually this includes a
check to see whether the garbage collector needs to be called. Without this
check, the garbage collector is called in loops like the one below only when the
system runs out of memory or triggers some other heuristic check.
At least, that's my understanding.
Setting all Javacript bugs to rginda QA Contact.
QA Contact: cbegle → rginda
Reporter | ||
Updated•24 years ago
|
Target Milestone: --- → Future
Comment 6•24 years ago
|
||
Reassigning Mike's JS Engine bugs to Patrick, as per recent meeting -
Assignee: mike+mozilla → beard
Status: ASSIGNED → NEW
Comment 7•23 years ago
|
||
xpcshell.c (at heart a clone of js.c) also lacks this. They can both be fixed
with the same chunk of code.
Comment 8•23 years ago
|
||
*** Bug 104099 has been marked as a duplicate of this bug. ***
Comment 9•21 years ago
|
||
Fixed on 11-Feb-2004 for Narcissus.
/be
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
Comment 10•20 years ago
|
||
testcase from bug 2235 based on example code from mff@research.att.com.
mff, do you mind if we include your testcase in the javascript test library?
Comment 11•20 years ago
|
||
Sure, use it! Wow, this is ancient history ;-)
Mary
(In reply to comment #10)
> Created an attachment (id=174881) [edit]
> js1_5/Regress/regress-3649-n.js
>
> testcase from bug 2235 based on example code from mff@research.att.com.
>
> mff, do you mind if we include your testcase in the javascript test library?
Comment 12•20 years ago
|
||
js1_5/Regress/regress-3649-n.js checked in
You need to log in
before you can comment on or make changes to this bug.
Description
•