Magic-Mirror/gecko-dev
1
0
Fork 0
mirror of https://github.com/mozilla/gecko-dev.git synced 2025-03-01 22:07:41 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Bug 1817767 - Use CopyChars for rope atomization cache. r=jandem

Browse Source 
Differential Revision: https://phabricator.services.mozilla.com/D170793
This commit is contained in:
Ted Campbell 2023-02-24 15:58:35 +00:00
parent c3b24d81c7
commit 596b90cea8
1 changed files with 3 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
js/src/vm/JSAtom.cpp
View File

@ -744,18 +744,15 @@ JSAtom* js::AtomizeString(JSContext* cx, JSString* str) {
JS::Latin1Char flattenRope[StringToAtomCache::MinStringLength];
mozilla::Maybe<StringToAtomCache::AtomTableKey> key;
size_t length = str->length();
if (!str->isLinear() && length < StringToAtomCache::MinStringLength &&
if (str->isRope() && length < StringToAtomCache::MinStringLength &&
str->hasLatin1Chars()) {
StringSegmentRange<StringToAtomCache::MinStringLength> iter(cx);
if (iter.init(str)) {
JS::AutoCheckCannotGC nogc;
size_t index = 0;
do {
const JSLinearString* s = iter.front();
size_t len = s->length();
const JS::Latin1Char* latinChars = s->latin1Chars(nogc);
memcpy(flattenRope + index, latinChars, len);
index += len;
CopyChars(flattenRope + index, *s);
index += s->length();
} while (iter.popFront() && !iter.empty());
if (JSAtom* atom = cx->caches().stringToAtomCache.lookupWithRopeChars(