Closed Bug 701097 Opened 13 years ago Closed 6 years ago

ArrayObject::try_splice marks deletedItems as non-simple more often than necessary

Categories

(Tamarin Graveyard :: Library, enhancement)

enhancement
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: pnkfelix, Unassigned)

References

Details

Forked off from Bug 688486, comment 32. It's referring to this bit of code in ArrayObject::try_splice if (this->m_canBeSimple) { [...] } else { // (easiest for now to just disable the simple path than // to try to dynamically determine when it is available) deletedItems->m_canBeSimple = false; deletedItems->m_lengthIfSimple = 0; } We're already performing sufficient work in this path that we could probably afford to check if the result is simple. Its just ugly to put that sort of hook directly into the AtomList splice method.
Tamarin is a dead project now. Mass WONTFIX.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → WONTFIX
Tamarin isn't maintained anymore. WONTFIX remaining bugs.
You need to log in before you can comment on or make changes to this bug.