roytam1/palemoon27
1
0
Fork 0
mirror of https://github.com/roytam1/palemoon27.git synced 2026-06-06 08:29:08 +00:00
Code Issues Projects Releases Wiki Activity

Be more careful about optimizing sorted arrays.

Browse Source
This commit is contained in:
Pale Moon
2017-03-14 11:20:03 +01:00
committed by roytam1
parent 4ec198f422
commit 0703663894
1 changed files with 8 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
js/src/jsarray.cpp
+8 -1
View File
@@ -1933,6 +1933,7 @@ js::array_sort(JSContext* cx, unsigned argc, Value* vp)
undefs = 0;
bool allStrings = true;
bool allInts = true;
bool extraIndexed = ObjectMayHaveExtraIndexedProperties(obj);
RootedValue v(cx);
for (uint32_t i = 0; i < len; i++) {
if (!CheckForInterrupt(cx))
@@ -1965,7 +1966,9 @@ js::array_sort(JSContext* cx, unsigned argc, Value* vp)
}
/* Here len == n + undefs + number_of_holes. */
bool defaultOrMatch;
if (fval.isNull()) {
defaultOrMatch = true;
/*
* Sort using the default comparator converting all elements to
* strings.
@@ -1989,6 +1992,7 @@ js::array_sort(JSContext* cx, unsigned argc, Value* vp)
if (comp == Match_Failure)
return false;
defaultOrMatch = comp != Match_None;
if (comp != Match_None) {
if (allInts) {
JS_ALWAYS_TRUE(vec.resize(n * 2));
@@ -2009,7 +2013,10 @@ js::array_sort(JSContext* cx, unsigned argc, Value* vp)
}
}
if (!InitArrayElements(cx, obj, 0, uint32_t(n), vec.begin(), DontUpdateTypes))
ShouldUpdateTypes updateTypes = !extraIndexed && (allStrings || allInts) && defaultOrMatch
? ShouldUpdateTypes::DontUpdateTypes
: ShouldUpdateTypes::UpdateTypes;
if (!InitArrayElements(cx, obj, 0, uint32_t(n), vec.begin(), updateTypes))
return false;
}