Ignore:
Timestamp:
Sep 13, 2012, 1:43:33 AM (13 years ago)
Author:
[email protected]
Message:

[Qt][Win] REGRESSION(r128400): It broke the build
​https://bugs.webkit.org/show_bug.cgi?id=96617

Patch by Filip Pizlo <​[email protected]> on 2012-09-13
Reviewed by Simon Hausmann.

Source/JavaScriptCore:

Changed "JSC::Array" to "JSC::ArrayClass" because it's not used often enough
for the brevity to be beneficial, and because "Array" causes too much namespace
pollution.

  • runtime/IndexingType.h:

(JSC):

  • runtime/JSArray.cpp:

(JSC::JSArray::pop):
(JSC::JSArray::push):
(JSC::JSArray::sortNumeric):
(JSC::JSArray::sort):
(JSC::JSArray::fillArgList):
(JSC::JSArray::copyToArguments):
(JSC::JSArray::compactForSorting):

  • runtime/JSObject.cpp:

(JSC::JSObject::getOwnPropertySlotByIndex):
(JSC::JSObject::putByIndex):
(JSC::JSObject::ensureArrayStorageExistsAndEnterDictionaryIndexingMode):
(JSC::JSObject::deletePropertyByIndex):
(JSC::JSObject::getOwnPropertyNames):
(JSC::JSObject::putByIndexBeyondVectorLength):
(JSC::JSObject::putDirectIndexBeyondVectorLength):
(JSC::JSObject::getNewVectorLength):
(JSC::JSObject::getOwnPropertyDescriptor):

  • runtime/JSObject.h:

(JSC::JSObject::getArrayLength):
(JSC::JSObject::getVectorLength):
(JSC::JSObject::canGetIndexQuickly):
(JSC::JSObject::canSetIndexQuickly):
(JSC::JSObject::inSparseIndexingMode):
(JSC::JSObject::ensureArrayStorage):

Source/WebCore:

  • bridge/runtime_array.h:

(JSC::RuntimeArray::createStructure):

Location:
trunk/Source/JavaScriptCore/runtime
Files:
4 edited

Legend:

Unmodified
Added
Removed

  • trunk/Source/JavaScriptCore/runtime/IndexingType.h

    r128400 r128428  
    4242static const IndexingType NonArray                 = 0;
    4343static const IndexingType NonArrayWithArrayStorage = HasArrayStorage;
    44 static const IndexingType Array                    = IsArray;
     44static const IndexingType ArrayClass               = IsArray; // I'd want to call this "Array" but this would lead to disastrous namespace pollution.
    4545static const IndexingType ArrayWithArrayStorage    = IsArray | HasArrayStorage;
    4646

  • trunk/Source/JavaScriptCore/runtime/JSArray.cpp

    r128400 r128428  
    403403   
    404404    switch (structure()->indexingType()) {
    405     case Array:
     405    case ArrayClass:
    406406        return jsUndefined();
    407407       
    … …  
    461461   
    462462    switch (structure()->indexingType()) {
    463     case Array: {
     463    case ArrayClass: {
    464464        putByIndexBeyondVectorLengthWithArrayStorage(exec, 0, value, true, createInitialArrayStorage(exec->globalData()));
    465465        break;
    … …  
    580580
    581581    switch (structure()->indexingType()) {
    582     case Array:
     582    case ArrayClass:
    583583        return;
    584584       
    … …  
    626626   
    627627    switch (structure()->indexingType()) {
    628     case Array:
     628    case ArrayClass:
    629629        return;
    630630       
    … …  
    792792   
    793793    switch (structure()->indexingType()) {
    794     case Array:
     794    case ArrayClass:
    795795        return;
    796796       
    … …  
    914914{
    915915    switch (structure()->indexingType()) {
    916     case Array:
     916    case ArrayClass:
    917917        return;
    918918   
    … …  
    944944    ASSERT(length == this->length());
    945945    switch (structure()->indexingType()) {
    946     case Array:
     946    case ArrayClass:
    947947        return;
    948948       
    … …  
    976976   
    977977    switch (structure()->indexingType()) {
    978     case Array:
     978    case ArrayClass:
    979979        return 0;
    980980

  • trunk/Source/JavaScriptCore/runtime/JSObject.cpp

    r128400 r128428  
    236236    switch (thisObject->structure()->indexingType()) {
    237237    case NonArray:
    238     case Array:
     238    case ArrayClass:
    239239        break;
    240240       
    … …  
    356356    switch (thisObject->structure()->indexingType()) {
    357357    case NonArray:
    358     case Array:
     358    case ArrayClass:
    359359        break;
    360360       
    … …  
    439439{
    440440    IndexingType oldType = structure()->indexingType();
    441     ASSERT_UNUSED(oldType, oldType == NonArray || oldType == Array);
     441    ASSERT_UNUSED(oldType, oldType == NonArray || oldType == ArrayClass);
    442442    Butterfly* newButterfly = m_butterfly->growArrayRight(
    443443        globalData, structure(), structure()->outOfLineCapacity(), false, 0,
    … …  
    472472        return enterDictionaryIndexingModeWhenArrayStorageAlreadyExists(globalData, m_butterfly->arrayStorage());
    473473       
    474     case Array:
     474    case ArrayClass:
    475475    case NonArray: {
    476476        createArrayStorage(globalData, 0, 0);
    … …  
    598598   
    599599    switch (thisObject->structure()->indexingType()) {
    600     case Array:
     600    case ArrayClass:
    601601    case NonArray:
    602602        return true;
    … …  
    764764    switch (object->structure()->indexingType()) {
    765765    case NonArray:
    766     case Array:
     766    case ArrayClass:
    767767        break;
    768768       
    … …  
    11911191    switch (structure()->indexingType()) {
    11921192    case NonArray:
    1193     case Array: {
     1193    case ArrayClass: {
    11941194        if (indexingShouldBeSparse()) {
    11951195            putByIndexBeyondVectorLengthWithArrayStorage(exec, i, value, shouldThrow, ensureArrayStorageExistsAndEnterDictionaryIndexingMode(globalData));
    … …  
    13001300    switch (structure()->indexingType()) {
    13011301    case NonArray:
    1302     case Array: {
     1302    case ArrayClass: {
    13031303        if (indexingShouldBeSparse() || attributes)
    13041304            return putDirectIndexBeyondVectorLengthWithArrayStorage(exec, i, value, attributes, mode, ensureArrayStorageExistsAndEnterDictionaryIndexingMode(globalData));
    … …  
    13561356    switch (structure()->indexingType()) {
    13571357    case NonArray:
    1358     case Array:
     1358    case ArrayClass:
    13591359        vectorLength = 0;
    13601360        length = 0;
    … …  
    14801480    switch (object->structure()->indexingType()) {
    14811481    case NonArray:
    1482     case Array:
     1482    case ArrayClass:
    14831483        return false;
    14841484       

  • trunk/Source/JavaScriptCore/runtime/JSObject.h

    r128400 r128428  
    137137            switch (structure()->indexingType()) {
    138138            case NonArray:
    139             case Array:
     139            case ArrayClass:
    140140                return 0;
    141141            case NonArrayWithArrayStorage:
    … …  
    152152            switch (structure()->indexingType()) {
    153153            case NonArray:
    154             case Array:
     154            case ArrayClass:
    155155                return 0;
    156156            case NonArrayWithArrayStorage:
    … …  
    191191            switch (structure()->indexingType()) {
    192192            case NonArray:
    193             case Array:
     193            case ArrayClass:
    194194                return false;
    195195            case NonArrayWithArrayStorage:
    … …  
    218218            switch (structure()->indexingType()) {
    219219            case NonArray:
    220             case Array:
     220            case ArrayClass:
    221221                return false;
    222222            case NonArrayWithArrayStorage:
    … …  
    300300            switch (structure()->indexingType()) {
    301301            case NonArray:
    302             case Array:
     302            case ArrayClass:
    303303                return false;
    304304            case NonArrayWithArrayStorage:
    … …  
    564564               
    565565            case NonArray:
    566             case Array:
     566            case ArrayClass:
    567567                return createInitialArrayStorage(globalData);
    568568               
Note: See TracChangeset for help on using the changeset viewer.