Changeset 58483 in webkit for trunk/JavaScriptCore/qt


Ignore:
Timestamp:
Apr 29, 2010, 1:14:50 AM (15 years ago)
Author:
[email protected]
Message:

2010-04-29 Jedrzej Nowacki <​[email protected]>

Reviewed by Simon Hausmann.

Small cleanup.

QScriptValuePrivate constructor shouldn't take QScriptEngine pointer
as a parameter, because it breaks a private / public separation rule.
QScriptEnginePrivate::get(QScriptEngine*) should return
a QScriptEnginePrivate pointer instead of a QScriptEnginePtr, it simplifies
implementation.

[Qt] QScriptValuePrivate class needs some cleanup.
​https://bugs.webkit.org/show_bug.cgi?id=37729

  • qt/api/qscriptengine_p.h: (QScriptEnginePrivate::get):
  • qt/api/qscriptvalue.cpp: (QScriptValue::QScriptValue):
  • qt/api/qscriptvalue_p.h: (QScriptValuePrivate::QScriptValuePrivate):
Location:
trunk/JavaScriptCore/qt/api
Files:
3 edited

Legend:

Unmodified
Added
Removed

  • trunk/JavaScriptCore/qt/api/qscriptengine_p.h

    r57637 r58483  
    3636class QScriptEnginePrivate : public QSharedData {
    3737public:
    38     static QScriptEnginePtr get(const QScriptEngine* q) { Q_ASSERT(q); return q->d_ptr; }
     38    static QScriptEnginePrivate* get(const QScriptEngine* q) { Q_ASSERT(q); return q->d_ptr.data(); }
    3939    static QScriptEngine* get(const QScriptEnginePrivate* d) { Q_ASSERT(d); return d->q_ptr; }
    4040

  • trunk/JavaScriptCore/qt/api/qscriptvalue.cpp

    r53850 r58483  
    116116*/
    117117QScriptValue::QScriptValue(QScriptEngine* engine, bool value)
    118     : d_ptr(new QScriptValuePrivate(engine, value))
    119 {
     118{
     119    if (engine)
     120        d_ptr = new QScriptValuePrivate(QScriptEnginePrivate::get(engine), value);
     121    else
     122        d_ptr = new QScriptValuePrivate(value);
    120123}
    121124
    … …  
    127130*/
    128131QScriptValue::QScriptValue(QScriptEngine* engine, int value)
    129     : d_ptr(new QScriptValuePrivate(engine, value))
    130 {
     132{
     133    if (engine)
     134        d_ptr = new QScriptValuePrivate(QScriptEnginePrivate::get(engine), value);
     135    else
     136        d_ptr = new QScriptValuePrivate(value);
    131137}
    132138
    … …  
    138144 */
    139145QScriptValue::QScriptValue(QScriptEngine* engine, uint value)
    140     : d_ptr(new QScriptValuePrivate(engine, value))
    141 {
     146{
     147    if (engine)
     148        d_ptr = new QScriptValuePrivate(QScriptEnginePrivate::get(engine), value);
     149    else
     150        d_ptr = new QScriptValuePrivate(value);
    142151}
    143152
    … …  
    149158*/
    150159QScriptValue::QScriptValue(QScriptEngine* engine, qsreal value)
    151     : d_ptr(new QScriptValuePrivate(engine, value))
    152 {
     160{
     161    if (engine)
     162        d_ptr = new QScriptValuePrivate(QScriptEnginePrivate::get(engine), value);
     163    else
     164        d_ptr = new QScriptValuePrivate(value);
    153165}
    154166
    … …  
    160172*/
    161173QScriptValue::QScriptValue(QScriptEngine* engine, const QString& value)
    162     : d_ptr(new QScriptValuePrivate(engine, value))
    163 {
     174{
     175    if (engine)
     176        d_ptr = new QScriptValuePrivate(QScriptEnginePrivate::get(engine), value);
     177    else
     178        d_ptr = new QScriptValuePrivate(value);
    164179}
    165180
    … …  
    171186*/
    172187QScriptValue::QScriptValue(QScriptEngine* engine, const char* value)
    173     : d_ptr(new QScriptValuePrivate(engine, QString::fromUtf8(value)))
    174 {
     188{
     189    if (engine)
     190        d_ptr = new QScriptValuePrivate(QScriptEnginePrivate::get(engine), QString::fromUtf8(value));
     191    else
     192        d_ptr = new QScriptValuePrivate(QString::fromUtf8(value));
    175193}
    176194
    … …  
    182200*/
    183201QScriptValue::QScriptValue(QScriptEngine* engine, SpecialValue value)
    184     : d_ptr(new QScriptValuePrivate(engine, value))
    185 {
     202{
     203    if (engine)
     204        d_ptr = new QScriptValuePrivate(QScriptEnginePrivate::get(engine), value);
     205    else
     206        d_ptr = new QScriptValuePrivate(value);
    186207}
    187208

  • trunk/JavaScriptCore/qt/api/qscriptvalue_p.h

    r57628 r58483  
    7777    inline QScriptValuePrivate(QScriptValue::SpecialValue value);
    7878
    79     inline QScriptValuePrivate(const QScriptEngine* engine, bool value);
    80     inline QScriptValuePrivate(const QScriptEngine* engine, int value);
    81     inline QScriptValuePrivate(const QScriptEngine* engine, uint value);
    82     inline QScriptValuePrivate(const QScriptEngine* engine, qsreal value);
    83     inline QScriptValuePrivate(const QScriptEngine* engine, const QString& value);
    84     inline QScriptValuePrivate(const QScriptEngine* engine, QScriptValue::SpecialValue value);
     79    inline QScriptValuePrivate(const QScriptEnginePrivate* engine, bool value);
     80    inline QScriptValuePrivate(const QScriptEnginePrivate* engine, int value);
     81    inline QScriptValuePrivate(const QScriptEnginePrivate* engine, uint value);
     82    inline QScriptValuePrivate(const QScriptEnginePrivate* engine, qsreal value);
     83    inline QScriptValuePrivate(const QScriptEnginePrivate* engine, const QString& value);
     84    inline QScriptValuePrivate(const QScriptEnginePrivate* engine, QScriptValue::SpecialValue value);
    8585
    8686    inline QScriptValuePrivate(const QScriptEnginePrivate* engine, JSValueRef value);
    … …  
    210210}
    211211
    212 QScriptValuePrivate::QScriptValuePrivate(const QScriptEngine* engine, bool value)
     212QScriptValuePrivate::QScriptValuePrivate(const QScriptEnginePrivate* engine, bool value)
    213213    : m_state(JSPrimitive)
    214 {
    215     if (!engine) {
    216         // slower path reinitialization
    217         m_state = CBool;
    218         m_number = value;
    219         m_value = 0;
    220     } else {
    221         m_engine = QScriptEnginePrivate::get(engine);
    222         m_value = m_engine->makeJSValue(value);
    223         JSValueProtect(context(), m_value);
    224     }
    225 }
    226 
    227 QScriptValuePrivate::QScriptValuePrivate(const QScriptEngine* engine, int value)
     214    , m_engine(const_cast<QScriptEnginePrivate*>(engine))
     215    , m_value(engine->makeJSValue(value))
     216{
     217    Q_ASSERT(engine);
     218    JSValueProtect(context(), m_value);
     219}
     220
     221QScriptValuePrivate::QScriptValuePrivate(const QScriptEnginePrivate* engine, int value)
    228222    : m_state(JSPrimitive)
    229 {
    230     if (!engine) {
    231         // slower path reinitialization
    232         m_state = CNumber;
    233         m_number = value;
    234         m_value = 0;
    235     } else {
    236         m_engine = QScriptEnginePrivate::get(engine);
    237         m_value = m_engine->makeJSValue(value);
    238         JSValueProtect(context(), m_value);
    239     }
    240 }
    241 
    242 QScriptValuePrivate::QScriptValuePrivate(const QScriptEngine* engine, uint value)
     223    , m_engine(const_cast<QScriptEnginePrivate*>(engine))
     224    , m_value(m_engine->makeJSValue(value))
     225{
     226    Q_ASSERT(engine);
     227    JSValueProtect(context(), m_value);
     228}
     229
     230QScriptValuePrivate::QScriptValuePrivate(const QScriptEnginePrivate* engine, uint value)
    243231    : m_state(JSPrimitive)
    244 {
    245     if (!engine) {
    246         // slower path reinitialization
    247         m_state = CNumber;
    248         m_number = value;
    249         m_value = 0;
    250     } else {
    251         m_engine = QScriptEnginePrivate::get(engine);
    252         m_value = m_engine->makeJSValue(value);
    253         JSValueProtect(context(), m_value);
    254     }
    255 }
    256 
    257 QScriptValuePrivate::QScriptValuePrivate(const QScriptEngine* engine, qsreal value)
     232    , m_engine(const_cast<QScriptEnginePrivate*>(engine))
     233    , m_value(m_engine->makeJSValue(value))
     234{
     235    Q_ASSERT(engine);
     236    JSValueProtect(context(), m_value);
     237}
     238
     239QScriptValuePrivate::QScriptValuePrivate(const QScriptEnginePrivate* engine, qsreal value)
    258240    : m_state(JSPrimitive)
    259 {
    260     if (!engine) {
    261         // slower path reinitialization
    262         m_state = CNumber;
    263         m_number = value;
    264         m_value = 0;
    265     } else {
    266         m_engine = QScriptEnginePrivate::get(engine);
    267         m_value = m_engine->makeJSValue(value);
    268         JSValueProtect(context(), m_value);
    269     }
    270 }
    271 
    272 QScriptValuePrivate::QScriptValuePrivate(const QScriptEngine* engine, const QString& value)
     241    , m_engine(const_cast<QScriptEnginePrivate*>(engine))
     242    , m_value(m_engine->makeJSValue(value))
     243{
     244    Q_ASSERT(engine);
     245    JSValueProtect(context(), m_value);
     246}
     247
     248QScriptValuePrivate::QScriptValuePrivate(const QScriptEnginePrivate* engine, const QString& value)
    273249    : m_state(JSPrimitive)
    274 {
    275     if (!engine) {
    276         // slower path reinitialization
    277         m_state = CString;
    278         m_string = value;
    279         m_value = 0;
    280     } else {
    281         m_engine = QScriptEnginePrivate::get(engine);
    282         m_value = m_engine->makeJSValue(value);
    283         JSValueProtect(context(), m_value);
    284     }
    285 }
    286 
    287 QScriptValuePrivate::QScriptValuePrivate(const QScriptEngine* engine, QScriptValue::SpecialValue value)
     250    , m_engine(const_cast<QScriptEnginePrivate*>(engine))
     251    , m_value(m_engine->makeJSValue(value))
     252{
     253    Q_ASSERT(engine);
     254    JSValueProtect(context(), m_value);
     255}
     256
     257QScriptValuePrivate::QScriptValuePrivate(const QScriptEnginePrivate* engine, QScriptValue::SpecialValue value)
    288258    : m_state(JSPrimitive)
    289 {
    290     if (!engine) {
    291         // slower path reinitialization
    292         m_state = CSpecial;
    293         m_number = value;
    294         m_value = 0;
    295     } else {
    296         m_engine = QScriptEnginePrivate::get(engine);
    297         m_value = m_engine->makeJSValue(value);
    298         JSValueProtect(context(), m_value);
    299     }
     259    , m_engine(const_cast<QScriptEnginePrivate*>(engine))
     260    , m_value(m_engine->makeJSValue(value))
     261{
     262    Q_ASSERT(engine);
     263    JSValueProtect(context(), m_value);
    300264}
    301265
    … …  
    306270{
    307271    Q_ASSERT(engine);
     272    Q_ASSERT(value);
    308273    JSValueProtect(context(), m_value);
    309274}
    … …  
    316281{
    317282    Q_ASSERT(engine);
     283    Q_ASSERT(value);
     284    Q_ASSERT(object);
    318285    JSValueProtect(context(), m_value);
    319286}
Note: See TracChangeset for help on using the changeset viewer.