Closed
Bug 435226
Opened 17 years ago
Closed 16 years ago
dense arrays yield bogus __count__ values
Categories
(Core :: JavaScript Engine, defect)
Core
JavaScript Engine
Tracking
()
RESOLVED
FIXED
People
(Reporter: crowderbt, Assigned: crowderbt)
References
Details
Attachments
(1 file)
|
932 bytes,
patch
|
shaver
:
review+
|
Details | Diff | Splinter Review |
+++ This bug was initially created as a clone of Bug #434837 +++
shavarrays are returning their length value instead of the actual count of elements; this is a bug in array_enumerate's handling of the INIT condition.
|
Assignee | |
Updated•17 years ago
|
Version: unspecified → Trunk
|
|
Assignee | |
Comment 1•17 years ago
|
||
This patch cannot be tested without the patch from bug 434837.
Attachment #322141 -
Flags: review?(shaver)
|
||
Comment 2•17 years ago
|
||
Comment on attachment 322141 [details] [diff] [review]
use COUNT instead of LENGTH
Does it test correctly on top of that patch?
|
|
Assignee | |
Comment 3•16 years ago
|
||
(In reply to comment #2)
> Does it test correctly on top of that patch?
Yes.
|
|
||
Comment 4•16 years ago
|
||
Comment on attachment 322141 [details] [diff] [review]
use COUNT instead of LENGTH
r=shaver
Attachment #322141 -
Flags: review?(shaver) → review+
|
|
Assignee | |
Comment 5•16 years ago
|
||
changeset: 15531:1bbd2daaabb6
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
|
||
Comment 6•16 years ago
|
||
see js1_5/extensions/regress-434837-01.js
Flags: in-testsuite+
Flags: in-litmus-
You need to log in
before you can comment on or make changes to this bug.
Description
•