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)
Tamarin Graveyard
Library
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.
Reporter | ||
Updated•13 years ago
|
Blocks: array-tracker
Comment 1•6 years ago
|
||
Tamarin is a dead project now. Mass WONTFIX.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → WONTFIX
Comment 2•6 years ago
|
||
Tamarin isn't maintained anymore. WONTFIX remaining bugs.
You need to log in
before you can comment on or make changes to this bug.
Description
•