diff options
Diffstat (limited to 'Source/JavaScriptCore/runtime/Lookup.cpp')
-rw-r--r-- | Source/JavaScriptCore/runtime/Lookup.cpp | 77 |
1 files changed, 44 insertions, 33 deletions
diff --git a/Source/JavaScriptCore/runtime/Lookup.cpp b/Source/JavaScriptCore/runtime/Lookup.cpp index dcee6ba58..a806dd052 100644 --- a/Source/JavaScriptCore/runtime/Lookup.cpp +++ b/Source/JavaScriptCore/runtime/Lookup.cpp @@ -1,5 +1,5 @@ /* - * Copyright (C) 2008, 2012, 2015 Apple Inc. All rights reserved. + * Copyright (C) 2008, 2012 Apple Inc. All rights reserved. * * This library is free software; you can redistribute it and/or * modify it under the terms of the GNU Lesser General Public @@ -21,34 +21,56 @@ #include "Lookup.h" #include "Executable.h" -#include "GetterSetter.h" #include "JSFunction.h" -#include "JSCInlines.h" +#include "Operations.h" namespace JSC { -void reifyStaticAccessor(VM& vm, const HashTableValue& value, JSObject& thisObj, PropertyName propertyName) +void HashTable::createTable(VM& vm) const { - JSGlobalObject* globalObject = thisObj.globalObject(); - GetterSetter* accessor = GetterSetter::create(vm, globalObject); - if (value.accessorGetter()) { - String getterName = WTF::tryMakeString(ASCIILiteral("get "), String(*propertyName.publicName())); - if (!getterName) - return; - accessor->setGetter(vm, globalObject, value.attributes() & Builtin - ? JSFunction::createBuiltinFunction(vm, value.builtinAccessorGetterGenerator()(vm), globalObject, getterName) - : JSFunction::create(vm, globalObject, 0, getterName, value.accessorGetter())); + ASSERT(!table); + int linkIndex = compactHashSizeMask + 1; + HashEntry* entries = new HashEntry[compactSize]; + for (int i = 0; i < compactSize; ++i) + entries[i].setKey(0); + for (int i = 0; values[i].key; ++i) { + StringImpl* identifier = Identifier::add(&vm, values[i].key).leakRef(); + int hashIndex = identifier->existingHash() & compactHashSizeMask; + HashEntry* entry = &entries[hashIndex]; + + if (entry->key()) { + while (entry->next()) { + entry = entry->next(); + } + ASSERT(linkIndex < compactSize); + entry->setNext(&entries[linkIndex++]); + entry = entry->next(); + } + + entry->initialize(identifier, values[i].attributes, values[i].value1, values[i].value2, values[i].intrinsic); } - thisObj.putDirectNonIndexAccessor(vm, propertyName, accessor, attributesForStructure(value.attributes())); + table = entries; } -bool setUpStaticFunctionSlot(ExecState* exec, const HashTableValue* entry, JSObject* thisObj, PropertyName propertyName, PropertySlot& slot) +void HashTable::deleteTable() const +{ + if (table) { + int max = compactSize; + for (int i = 0; i != max; ++i) { + if (StringImpl* key = table[i].key()) + key->deref(); + } + delete [] table; + table = 0; + } +} + +bool setUpStaticFunctionSlot(ExecState* exec, const HashEntry* entry, JSObject* thisObj, PropertyName propertyName, PropertySlot& slot) { ASSERT(thisObj->globalObject()); - ASSERT(entry->attributes() & BuiltinOrFunctionOrAccessor); + ASSERT(entry->attributes() & Function); VM& vm = exec->vm(); unsigned attributes; - bool isAccessor = entry->attributes() & Accessor; PropertyOffset offset = thisObj->getDirectOffset(vm, propertyName, attributes); if (!isValidOffset(offset)) { @@ -56,26 +78,15 @@ bool setUpStaticFunctionSlot(ExecState* exec, const HashTableValue* entry, JSObj // all static functions at that time - after this we shouldn't be re-adding anything. if (thisObj->staticFunctionsReified()) return false; - - if (entry->attributes() & Builtin) - thisObj->putDirectBuiltinFunction(vm, thisObj->globalObject(), propertyName, entry->builtinGenerator()(vm), attributesForStructure(entry->attributes())); - else if (entry->attributes() & Function) { - thisObj->putDirectNativeFunction( - vm, thisObj->globalObject(), propertyName, entry->functionLength(), - entry->function(), entry->intrinsic(), attributesForStructure(entry->attributes())); - } else { - ASSERT(isAccessor); - reifyStaticAccessor(vm, *entry, *thisObj, propertyName); - } - + + thisObj->putDirectNativeFunction( + vm, thisObj->globalObject(), propertyName, entry->functionLength(), + entry->function(), entry->intrinsic(), entry->attributes()); offset = thisObj->getDirectOffset(vm, propertyName, attributes); ASSERT(isValidOffset(offset)); } - if (isAccessor) - slot.setCacheableGetterSlot(thisObj, attributes, jsCast<GetterSetter*>(thisObj->getDirect(offset)), offset); - else - slot.setValue(thisObj, attributes, thisObj->getDirect(offset), offset); + slot.setValue(thisObj, attributes, thisObj->getDirect(offset), offset); return true; } |