5 files changed, 18 insertions, 15 deletions
diff --git a/src/qml/doc/src/qmllanguageref/typesystem/basictypes.qdoc b/src/qml/doc/src/qmllanguageref/typesystem/basictypes.qdoc
index c4c1b61693..5144fe219e 100644
--- a/src/qml/doc/src/qmllanguageref/typesystem/basictypes.qdoc
+++ b/src/qml/doc/src/qmllanguageref/typesystem/basictypes.qdoc
@@ -44,6 +44,10 @@ Basic types can be used to refer to:
 \li A value that contains a simple set of property-value pairs (e.g. \l size refers to a value with \c width and \c height attributes)
 \endlist
 
+When a variable or property holds a basic type and it is assigned to another
+variable or property, then a copy of the value is made. In JavaScript, this
+value is called a primitive value.
+
 \sa {qtqml-typesystem-topic.html}{The QML Type System}
 
 
diff --git a/src/qml/jsruntime/qv4memberdata.cpp b/src/qml/jsruntime/qv4memberdata.cpp
index f327c85001..ffebe1b5da 100644
--- a/src/qml/jsruntime/qv4memberdata.cpp
+++ b/src/qml/jsruntime/qv4memberdata.cpp
@@ -72,8 +72,8 @@ Heap::MemberData *MemberData::allocate(ExecutionEngine *e, uint n, Heap::MemberD
 
     // The above code can overflow in a number of interesting ways. All of those are unsigned,
     // and therefore defined behavior. Still, apply some sane bounds.
-    if (alloc > std::numeric_limits<int>::max())
-        alloc = std::numeric_limits<int>::max();
+    if (alloc > size_t(std::numeric_limits<int>::max()))
+        alloc = size_t(std::numeric_limits<int>::max());
 
     Heap::MemberData *m;
     if (old) {
diff --git a/src/qml/jsruntime/qv4object.cpp b/src/qml/jsruntime/qv4object.cpp
index 206b410cf4..89161433ed 100644
--- a/src/qml/jsruntime/qv4object.cpp
+++ b/src/qml/jsruntime/qv4object.cpp
@@ -93,7 +93,7 @@ void Heap::Object::setUsedAsProto()
     internalClass.set(internalClass->engine, internalClass->asProtoClass());
 }
 
-ReturnedValue Object::getValueAccessor(const Value &thisObject, const Value &v, PropertyAttributes attrs)
+ReturnedValue Object::getValueAccessor(const Value *thisObject, const Value &v, PropertyAttributes attrs)
 {
     if (!attrs.isAccessor())
         return v.asReturnedValue();
@@ -103,7 +103,8 @@ ReturnedValue Object::getValueAccessor(const Value &thisObject, const Value &v,
 
     Scope scope(f->engine());
     JSCallData jsCallData(scope);
-    *jsCallData->thisObject = thisObject;
+    if (thisObject)
+        *jsCallData->thisObject = *thisObject;
     return f->call(jsCallData);
 }
 
@@ -415,7 +416,7 @@ ReturnedValue Object::internalGet(PropertyKey id, const Value *receiver, bool *h
             if (o->arrayData && o->arrayData->getProperty(index, pd, &attrs)) {
                 if (hasProperty)
                     *hasProperty = true;
-                return Object::getValue(*receiver, pd->value, attrs);
+                return Object::getValue(receiver, pd->value, attrs);
             }
             if (o->internalClass->vtable->type == Type_StringObject) {
                 ScopedString str(scope, static_cast<Heap::StringObject *>(o)->getIndex(index));
@@ -436,7 +437,7 @@ ReturnedValue Object::internalGet(PropertyKey id, const Value *receiver, bool *h
             if (idx.isValid()) {
                 if (hasProperty)
                     *hasProperty = true;
-                return Object::getValue(*receiver, *o->propertyData(idx.index), idx.attrs);
+                return Object::getValue(receiver, *o->propertyData(idx.index), idx.attrs);
             }
             o = o->prototype();
             if (!o || o->internalClass->vtable->get != Object::virtualGet)
diff --git a/src/qml/jsruntime/qv4object_p.h b/src/qml/jsruntime/qv4object_p.h
index 38055ef407..52ad3ea319 100644
--- a/src/qml/jsruntime/qv4object_p.h
+++ b/src/qml/jsruntime/qv4object_p.h
@@ -185,22 +185,22 @@ struct Q_QML_EXPORT Object: Managed {
     //
     // helpers
     //
-    static ReturnedValue getValue(const Value &thisObject, const Value &v, PropertyAttributes attrs) {
+    static ReturnedValue getValue(const Value *thisObject, const Value &v, PropertyAttributes attrs) {
         if (attrs.isData())
             return v.asReturnedValue();
         return getValueAccessor(thisObject, v, attrs);
     }
     ReturnedValue getValue(const Value &v, PropertyAttributes attrs) const {
-        return getValue(*this, v, attrs);
+        return getValue(this, v, attrs);
     }
     ReturnedValue getValueByIndex(uint propertyIndex) const {
         PropertyAttributes attrs = internalClass()->propertyData.at(propertyIndex);
         const Value *v = propertyData(propertyIndex);
         if (!attrs.isAccessor())
             return v->asReturnedValue();
-        return getValueAccessor(*this, *v, attrs);
+        return getValueAccessor(this, *v, attrs);
     }
-    static ReturnedValue getValueAccessor(const Value &thisObject, const Value &v, PropertyAttributes attrs);
+    static ReturnedValue getValueAccessor(const Value *thisObject, const Value &v, PropertyAttributes attrs);
 
     bool putValue(uint memberIndex, PropertyAttributes attrs, const Value &value);
 
diff --git a/src/qml/qml/qqmlengine_p.h b/src/qml/qml/qqmlengine_p.h
index 668dfed91e..ffe0e36d75 100644
--- a/src/qml/qml/qqmlengine_p.h
+++ b/src/qml/qml/qqmlengine_p.h
@@ -311,8 +311,8 @@ Returns true if the calling thread is the QQmlEngine thread.
 */
 bool QQmlEnginePrivate::isEngineThread() const
 {
-    Q_Q(const QQmlEngine);
-    return QThread::currentThread() == q->thread();
+
+    return QThread::currentThread() == q_ptr->thread();
 }
 
 /*!
@@ -337,8 +337,6 @@ the instance directly if not.
 template<typename T>
 void QQmlEnginePrivate::deleteInEngineThread(T *value)
 {
-    Q_Q(QQmlEngine);
-
     Q_ASSERT(value);
     if (isEngineThread()) {
         delete value;
@@ -354,7 +352,7 @@ void QQmlEnginePrivate::deleteInEngineThread(T *value)
         toDeleteInEngineThread.append(i);
         mutex.unlock();
         if (wasEmpty)
-            QCoreApplication::postEvent(q, new QEvent(QEvent::User));
+            QCoreApplication::postEvent(q_ptr, new QEvent(QEvent::User));
     }
 }