Closed Bug 1508346 Opened Last year Closed Last year

Consider having a ListObject class

Categories

(Core :: JavaScript Engine, enhancement)

enhancement
Not set

Tracking

()

RESOLVED FIXED
mozilla65
Tracking Status
firefox65 --- fixed

People

(Reporter: jorendorff, Assigned: jorendorff)

Details

Attachments

(1 file, 1 obsolete file)

I wrote a patch for this that it turns out I didn't need.

I don't usually go for this sort of thing, because it's about +50 lines of code net, without actually doing anything different. But perhaps there's a safety argument for doing it anyway. Punting the decision to tcampbell.

Calling this ListObject is a bit misleading; standards use "List" even for things like the entries of a Map or Set, where of course we other data structures. ListObject is just our fallback. I tried to explain this in a comment, but of course that only works to the extent people read it...
Depends on D12323
Attachment #9027022 - Attachment description: Bug 1508346 - Add class ListObject. r?tcampbell → Bug 1508346 - Add class ListObject. r=jandem
Attachment #9027022 - Attachment is obsolete: true
https://hg.mozilla.org/mozilla-central/rev/ac1c8b26e690
Status: NEW → RESOLVED
Closed: Last year
Resolution: --- → FIXED
Target Milestone: --- → mozilla65
Assignee: nobody → jorendorff
You need to log in before you can comment on or make changes to this bug.