Changeset 184972 in webkit


Ignore:
Timestamp:
May 28, 2015, 6:08:06 PM (10 years ago)
Author:
Gyuyoung Kim
Message:

Purge PassRefPtr in StringConcatenate.h
​https://bugs.webkit.org/show_bug.cgi?id=145376

Reviewed by Darin Adler.

Replace PassRefPtr with RefPtr. Additionally use nullptr instead of 0.

  • wtf/text/StringConcatenate.h:

(WTF::tryMakeString):
(WTF::makeString):

Location:
trunk/Source/WTF
Files:
2 edited

Legend:

Unmodified
Added
Removed

  • trunk/Source/WTF/ChangeLog

    r184949 r184972  
     12015-05-28  Gyuyoung Kim  <gyuyoung.kim@webkit.org>
     2
     3        Purge PassRefPtr in StringConcatenate.h
     4        https://bugs.webkit.org/show_bug.cgi?id=145376
     5
     6        Reviewed by Darin Adler.
     7
     8        Replace PassRefPtr with RefPtr. Additionally use nullptr instead of 0.
     9
     10        * wtf/text/StringConcatenate.h:
     11        (WTF::tryMakeString):
     12        (WTF::makeString):
     13
    1142015-05-28  Zan Dobersek  <zdobersek@igalia.com>
    215

  • trunk/Source/WTF/wtf/text/StringConcatenate.h

    r174255 r184972  
    257257
    258258template<typename StringType1, typename StringType2>
    259 PassRefPtr<StringImpl> tryMakeString(StringType1 string1, StringType2 string2)
     259RefPtr<StringImpl> tryMakeString(StringType1 string1, StringType2 string2)
    260260{
    261261    StringTypeAdapter<StringType1> adapter1(string1);
    … …  
    266266    sumWithOverflow(length, adapter2.length(), overflow);
    267267    if (overflow)
    268         return 0;
     268        return nullptr;
    269269
    270270    if (adapter1.is8Bit() && adapter2.is8Bit()) {
    … …  
    272272        RefPtr<StringImpl> resultImpl = StringImpl::tryCreateUninitialized(length, buffer);
    273273        if (!resultImpl)
    274             return 0;
     274            return nullptr;
    275275
    276276        LChar* result = buffer;
    … …  
    279279        adapter2.writeTo(result);
    280280
    281         return resultImpl.release();
     281        return WTF::move(resultImpl);
    282282    }
    283283
    … …  
    285285    RefPtr<StringImpl> resultImpl = StringImpl::tryCreateUninitialized(length, buffer);
    286286    if (!resultImpl)
    287         return 0;
     287        return nullptr;
    288288
    289289    UChar* result = buffer;
    … …  
    292292    adapter2.writeTo(result);
    293293
    294     return resultImpl.release();
     294    return WTF::move(resultImpl);
    295295}
    296296
    297297template<typename StringType1, typename StringType2, typename StringType3>
    298 PassRefPtr<StringImpl> tryMakeString(StringType1 string1, StringType2 string2, StringType3 string3)
     298RefPtr<StringImpl> tryMakeString(StringType1 string1, StringType2 string2, StringType3 string3)
    299299{
    300300    StringTypeAdapter<StringType1> adapter1(string1);
    … …  
    307307    sumWithOverflow(length, adapter3.length(), overflow);
    308308    if (overflow)
    309         return 0;
     309        return nullptr;
    310310
    311311    if (adapter1.is8Bit() && adapter2.is8Bit() && adapter3.is8Bit()) {
    … …  
    313313        RefPtr<StringImpl> resultImpl = StringImpl::tryCreateUninitialized(length, buffer);
    314314        if (!resultImpl)
    315             return 0;
     315            return nullptr;
    316316
    317317        LChar* result = buffer;
    … …  
    322322        adapter3.writeTo(result);
    323323
    324         return resultImpl.release();
     324        return WTF::move(resultImpl);
    325325    }
    326326
    … …  
    328328    RefPtr<StringImpl> resultImpl = StringImpl::tryCreateUninitialized(length, buffer);
    329329    if (!resultImpl)
    330         return 0;
     330        return nullptr;
    331331
    332332    UChar* result = buffer;
    … …  
    337337    adapter3.writeTo(result);
    338338
    339     return resultImpl.release();
     339    return WTF::move(resultImpl);
    340340}
    341341
    342342template<typename StringType1, typename StringType2, typename StringType3, typename StringType4>
    343 PassRefPtr<StringImpl> tryMakeString(StringType1 string1, StringType2 string2, StringType3 string3, StringType4 string4)
     343RefPtr<StringImpl> tryMakeString(StringType1 string1, StringType2 string2, StringType3 string3, StringType4 string4)
    344344{
    345345    StringTypeAdapter<StringType1> adapter1(string1);
    … …  
    354354    sumWithOverflow(length, adapter4.length(), overflow);
    355355    if (overflow)
    356         return 0;
     356        return nullptr;
    357357
    358358    if (adapter1.is8Bit() && adapter2.is8Bit() && adapter3.is8Bit() && adapter4.is8Bit()) {
    … …  
    360360        RefPtr<StringImpl> resultImpl = StringImpl::tryCreateUninitialized(length, buffer);
    361361        if (!resultImpl)
    362             return 0;
     362            return nullptr;
    363363
    364364        LChar* result = buffer;
    … …  
    371371        adapter4.writeTo(result);
    372372
    373         return resultImpl.release();
     373        return WTF::move(resultImpl);
    374374    }
    375375
    … …  
    377377    RefPtr<StringImpl> resultImpl = StringImpl::tryCreateUninitialized(length, buffer);
    378378    if (!resultImpl)
    379         return 0;
     379        return nullptr;
    380380
    381381    UChar* result = buffer;
    … …  
    388388    adapter4.writeTo(result);
    389389
    390     return resultImpl.release();
     390    return WTF::move(resultImpl);
    391391}
    392392
    393393template<typename StringType1, typename StringType2, typename StringType3, typename StringType4, typename StringType5>
    394 PassRefPtr<StringImpl> tryMakeString(StringType1 string1, StringType2 string2, StringType3 string3, StringType4 string4, StringType5 string5)
     394RefPtr<StringImpl> tryMakeString(StringType1 string1, StringType2 string2, StringType3 string3, StringType4 string4, StringType5 string5)
    395395{
    396396    StringTypeAdapter<StringType1> adapter1(string1);
    … …  
    407407    sumWithOverflow(length, adapter5.length(), overflow);
    408408    if (overflow)
    409         return 0;
     409        return nullptr;
    410410
    411411    if (adapter1.is8Bit() && adapter2.is8Bit() && adapter3.is8Bit() && adapter4.is8Bit() && adapter5.is8Bit()) {
    … …  
    413413        RefPtr<StringImpl> resultImpl = StringImpl::tryCreateUninitialized(length, buffer);
    414414        if (!resultImpl)
    415             return 0;
     415            return nullptr;
    416416
    417417        LChar* result = buffer;
    … …  
    426426        adapter5.writeTo(result);
    427427
    428         return resultImpl.release();
     428        return WTF::move(resultImpl);
    429429    }
    430430
    … …  
    432432    RefPtr<StringImpl> resultImpl = StringImpl::tryCreateUninitialized(length, buffer);
    433433    if (!resultImpl)
    434         return 0;
     434        return nullptr;
    435435
    436436    UChar* result = buffer;
    … …  
    445445    adapter5.writeTo(result);
    446446
    447     return resultImpl.release();
     447    return WTF::move(resultImpl);
    448448}
    449449
    450450template<typename StringType1, typename StringType2, typename StringType3, typename StringType4, typename StringType5, typename StringType6>
    451 PassRefPtr<StringImpl> tryMakeString(StringType1 string1, StringType2 string2, StringType3 string3, StringType4 string4, StringType5 string5, StringType6 string6)
     451RefPtr<StringImpl> tryMakeString(StringType1 string1, StringType2 string2, StringType3 string3, StringType4 string4, StringType5 string5, StringType6 string6)
    452452{
    453453    StringTypeAdapter<StringType1> adapter1(string1);
    … …  
    466466    sumWithOverflow(length, adapter6.length(), overflow);
    467467    if (overflow)
    468         return 0;
     468        return nullptr;
    469469
    470470    if (adapter1.is8Bit() && adapter2.is8Bit() && adapter3.is8Bit() && adapter4.is8Bit() && adapter5.is8Bit() && adapter6.is8Bit()) {
    … …  
    472472        RefPtr<StringImpl> resultImpl = StringImpl::tryCreateUninitialized(length, buffer);
    473473        if (!resultImpl)
    474             return 0;
     474            return nullptr;
    475475
    476476        LChar* result = buffer;
    … …  
    487487        adapter6.writeTo(result);
    488488
    489         return resultImpl.release();       
     489        return WTF::move(resultImpl);
    490490    }
    491491
    … …  
    493493    RefPtr<StringImpl> resultImpl = StringImpl::tryCreateUninitialized(length, buffer);
    494494    if (!resultImpl)
    495         return 0;
     495        return nullptr;
    496496
    497497    UChar* result = buffer;
    … …  
    508508    adapter6.writeTo(result);
    509509
    510     return resultImpl.release();
     510    return WTF::move(resultImpl);
    511511}
    512512
    513513template<typename StringType1, typename StringType2, typename StringType3, typename StringType4, typename StringType5, typename StringType6, typename StringType7>
    514 PassRefPtr<StringImpl> tryMakeString(StringType1 string1, StringType2 string2, StringType3 string3, StringType4 string4, StringType5 string5, StringType6 string6, StringType7 string7)
     514RefPtr<StringImpl> tryMakeString(StringType1 string1, StringType2 string2, StringType3 string3, StringType4 string4, StringType5 string5, StringType6 string6, StringType7 string7)
    515515{
    516516    StringTypeAdapter<StringType1> adapter1(string1);
    … …  
    531531    sumWithOverflow(length, adapter7.length(), overflow);
    532532    if (overflow)
    533         return 0;
     533        return nullptr;
    534534
    535535    if (adapter1.is8Bit() && adapter2.is8Bit() && adapter3.is8Bit() && adapter4.is8Bit() && adapter5.is8Bit() && adapter6.is8Bit() && adapter7.is8Bit()) {
    … …  
    537537        RefPtr<StringImpl> resultImpl = StringImpl::tryCreateUninitialized(length, buffer);
    538538        if (!resultImpl)
    539             return 0;
     539            return nullptr;
    540540
    541541        LChar* result = buffer;
    … …  
    554554        adapter7.writeTo(result);
    555555
    556         return resultImpl.release();
     556        return WTF::move(resultImpl);
    557557    }
    558558
    … …  
    560560    RefPtr<StringImpl> resultImpl = StringImpl::tryCreateUninitialized(length, buffer);
    561561    if (!resultImpl)
    562         return 0;
     562        return nullptr;
    563563
    564564    UChar* result = buffer;
    … …  
    577577    adapter7.writeTo(result);
    578578
    579     return resultImpl.release();
     579    return WTF::move(resultImpl);
    580580}
    581581
    582582template<typename StringType1, typename StringType2, typename StringType3, typename StringType4, typename StringType5, typename StringType6, typename StringType7, typename StringType8>
    583 PassRefPtr<StringImpl> tryMakeString(StringType1 string1, StringType2 string2, StringType3 string3, StringType4 string4, StringType5 string5, StringType6 string6, StringType7 string7, StringType8 string8)
     583RefPtr<StringImpl> tryMakeString(StringType1 string1, StringType2 string2, StringType3 string3, StringType4 string4, StringType5 string5, StringType6 string6, StringType7 string7, StringType8 string8)
    584584{
    585585    StringTypeAdapter<StringType1> adapter1(string1);
    … …  
    602602    sumWithOverflow(length, adapter8.length(), overflow);
    603603    if (overflow)
    604         return 0;
     604        return nullptr;
    605605
    606606    if (adapter1.is8Bit() && adapter2.is8Bit() && adapter3.is8Bit() && adapter4.is8Bit() && adapter5.is8Bit() && adapter6.is8Bit() && adapter7.is8Bit() && adapter8.is8Bit()) {
    … …  
    608608        RefPtr<StringImpl> resultImpl = StringImpl::tryCreateUninitialized(length, buffer);
    609609        if (!resultImpl)
    610             return 0;
     610            return nullptr;
    611611
    612612        LChar* result = buffer;
    … …  
    627627        adapter8.writeTo(result);
    628628
    629         return resultImpl.release();
     629        return WTF::move(resultImpl);
    630630    }
    631631
    … …  
    633633    RefPtr<StringImpl> resultImpl = StringImpl::tryCreateUninitialized(length, buffer);
    634634    if (!resultImpl)
    635         return 0;
     635        return nullptr;
    636636
    637637    UChar* result = buffer;
    … …  
    652652    adapter8.writeTo(result);
    653653
    654     return resultImpl.release();
     654    return WTF::move(resultImpl);
    655655}
    656656
    657657template<typename StringType1, typename StringType2, typename StringType3, typename StringType4, typename StringType5, typename StringType6, typename StringType7, typename StringType8, typename StringType9>
    658 PassRefPtr<StringImpl> tryMakeString(StringType1 string1, StringType2 string2, StringType3 string3, StringType4 string4, StringType5 string5, StringType6 string6, StringType7 string7, StringType8 string8, StringType9 string9)
     658RefPtr<StringImpl> tryMakeString(StringType1 string1, StringType2 string2, StringType3 string3, StringType4 string4, StringType5 string5, StringType6 string6, StringType7 string7, StringType8 string8, StringType9 string9)
    659659{
    660660    StringTypeAdapter<StringType1> adapter1(string1);
    … …  
    679679    sumWithOverflow(length, adapter9.length(), overflow);
    680680    if (overflow)
    681         return 0;
     681        return nullptr;
    682682
    683683    if (adapter1.is8Bit() && adapter2.is8Bit() && adapter3.is8Bit() && adapter4.is8Bit() && adapter5.is8Bit() && adapter6.is8Bit() && adapter7.is8Bit() && adapter8.is8Bit() && adapter9.is8Bit()) {
    … …  
    685685        RefPtr<StringImpl> resultImpl = StringImpl::tryCreateUninitialized(length, buffer);
    686686        if (!resultImpl)
    687             return 0;
     687            return nullptr;
    688688
    689689        LChar* result = buffer;
    … …  
    706706        adapter9.writeTo(result);
    707707
    708         return resultImpl.release();
     708        return WTF::move(resultImpl);
    709709    }
    710710
    … …  
    712712    RefPtr<StringImpl> resultImpl = StringImpl::tryCreateUninitialized(length, buffer);
    713713    if (!resultImpl)
    714         return 0;
     714        return nullptr;
    715715
    716716    UChar* result = buffer;
    … …  
    733733    adapter9.writeTo(result);
    734734
    735     return resultImpl.release();
     735    return WTF::move(resultImpl);
    736736}
    737737
    … …  
    750750    if (!resultImpl)
    751751        CRASH();
    752     return resultImpl.release();
     752    return WTF::move(resultImpl);
    753753}
    754754
    … …  
    759759    if (!resultImpl)
    760760        CRASH();
    761     return resultImpl.release();
     761    return WTF::move(resultImpl);
    762762}
    763763
    … …  
    768768    if (!resultImpl)
    769769        CRASH();
    770     return resultImpl.release();
     770    return WTF::move(resultImpl);
    771771}
    772772
    … …  
    777777    if (!resultImpl)
    778778        CRASH();
    779     return resultImpl.release();
     779    return WTF::move(resultImpl);
    780780}
    781781
    … …  
    786786    if (!resultImpl)
    787787        CRASH();
    788     return resultImpl.release();
     788    return WTF::move(resultImpl);
    789789}
    790790
    … …  
    795795    if (!resultImpl)
    796796        CRASH();
    797     return resultImpl.release();
     797    return WTF::move(resultImpl);
    798798}
    799799
    … …  
    804804    if (!resultImpl)
    805805        CRASH();
    806     return resultImpl.release();
     806    return WTF::move(resultImpl);
    807807}
    808808
    … …  
    813813    if (!resultImpl)
    814814        CRASH();
    815     return resultImpl.release();
     815    return WTF::move(resultImpl);
    816816}
    817817
Note: See TracChangeset for help on using the changeset viewer.