aboutsummaryrefslogtreecommitdiffstats
path: root/libpyside
diff options
context:
space:
mode:
authorHugo Parente Lima <hugo.pl@gmail.com>2011-10-26 18:28:51 -0200
committerHugo Parente Lima <hugo.pl@gmail.com>2012-03-08 16:56:15 -0300
commit15ceed791f8dda4aa1256e67c11f8a748ce4b7b0 (patch)
treeb71a2c970c244e586de94ddfcb6f5e35e12d27ee /libpyside
parentf55d088d4449f0a4b77ab428442a344790fbecd2 (diff)
Fix bug 1016 - "Calling of Q_INVOKABLE method returning not QVariant is impossible..."
Diffstat (limited to 'libpyside')
-rw-r--r--libpyside/pysidemetafunction.cpp72
1 files changed, 55 insertions, 17 deletions
diff --git a/libpyside/pysidemetafunction.cpp b/libpyside/pysidemetafunction.cpp
index 9338f8eda..de4d3244a 100644
--- a/libpyside/pysidemetafunction.cpp
+++ b/libpyside/pysidemetafunction.cpp
@@ -97,31 +97,69 @@ void functionFree(void *self)
PyObject* functionCall(PyObject* self, PyObject* args, PyObject* kw)
{
- static Shiboken::TypeResolver* typeResolver = Shiboken::TypeResolver::get("QVariant");
- Q_ASSERT(typeResolver);
-
- QGenericArgument gArgs[10];
- QVariant vArgs[10];
PySideMetaFunction* function = reinterpret_cast<PySideMetaFunction*>(self);
QMetaMethod method = function->d->method;
- int argsGiven = method.parameterTypes().size();
+ QList<QByteArray> argTypes = method.parameterTypes();
+
+ // args given plus return type
+ int numArgs = PyTuple_GET_SIZE(args) + 1;
+
+ if (numArgs - 1 != argTypes.count()) {
+ PyErr_Format(PyExc_TypeError, "%s only accepts %d arguments, %d given!", method.signature(), argTypes.count(), numArgs);
+ return 0;
+ }
- for (int i = 0; i < argsGiven; ++i) {
- Shiboken::AutoDecRef pyArg(PySequence_GetItem(args, i));
- gArgs[i] = Q_ARG(QVariant, vArgs[i]);
- void* v[1] = { &vArgs[i] };
- typeResolver->toCpp(pyArg, v);
+ QVariant* methValues = new QVariant[numArgs];
+ void** methArgs = new void*[numArgs];
+
+ // Prepare room for return type
+ const char* returnType = method.typeName();
+ if (returnType)
+ argTypes.prepend(returnType);
+ else
+ argTypes.prepend(QByteArray());
+
+ int i;
+ for (i = 0; i < numArgs; ++i) {
+ const QByteArray& typeName = argTypes[i];
+ // This must happen only when the method hasn't return type.
+ if (typeName.isEmpty()) {
+ methArgs[i] = 0;
+ continue;
+ }
+
+ Shiboken::TypeResolver* typeResolver = Shiboken::TypeResolver::get(typeName);
+ if (typeResolver) {
+ if (Shiboken::TypeResolver::getType(typeName) == Shiboken::TypeResolver::ValueType) {
+ int typeId = QMetaType::type(typeName);
+ if (!typeId) {
+ PyErr_Format(PyExc_TypeError, "Value type used on signal needs to be registered on meta type: %s", typeName.data());
+ break;
+ }
+ methValues[i] = QVariant(typeId, (void*) 0);
+ }
+ methArgs[i] = methValues[i].data();
+ if (i != 0) // Don't do this for return type
+ typeResolver->toCpp(PyTuple_GET_ITEM(args, i - 1), &methArgs[i]);
+ } else {
+ PyErr_Format(PyExc_TypeError, "Unknown type used to emit a signal: %s", argTypes[i].constData());
+ break;
+ }
}
- QVariant retVariant;
- QGenericReturnArgument returnValue = Q_RETURN_ARG(QVariant, retVariant);
- method.invoke(function->d->qobject, returnValue, gArgs[0], gArgs[1], gArgs[2], gArgs[3], gArgs[4], gArgs[5], gArgs[6], gArgs[7], gArgs[8], gArgs[9]);
+ bool ok = i == numArgs;
+ if (ok)
+ QMetaObject::metacall(function->d->qobject, QMetaObject::InvokeMetaMethod, method.methodIndex(), methArgs);
+
+ static Shiboken::TypeResolver* qVariantTypeResolver = Shiboken::TypeResolver::get("QVariant");
+ Q_ASSERT(qVariantTypeResolver);
+ PyObject* retVal = qVariantTypeResolver->toPython(&methValues[0]);
- if (retVariant.isValid())
- return typeResolver->toPython(&retVariant);
+ delete[] methArgs;
+ delete[] methValues;
- Py_RETURN_NONE;
+ return retVal;
}
} // extern "C"