diff options
author | Hugo Parente Lima <hugo.lima@openbossa.org> | 2010-04-12 18:35:21 -0300 |
---|---|---|
committer | Hugo Parente Lima <hugo.lima@openbossa.org> | 2010-04-12 18:37:20 -0300 |
commit | 27458bcd2f87e29dbdb60ba0de463261fedd3afa (patch) | |
tree | 178a145bbc629e903a64765977d9416e716c6c49 /libpyside/signalmanager.cpp | |
parent | f4c941bdadd8fd0ea4286745551af55899b76494 (diff) |
Fix bug#197, among some reference leaks.
Reviewer: Renato Araújo <renato.filho@openbossa.org>
Reviewer: Luciano Wolf <luciano.wolf@openbossa.org>
Diffstat (limited to 'libpyside/signalmanager.cpp')
-rw-r--r-- | libpyside/signalmanager.cpp | 28 |
1 files changed, 19 insertions, 9 deletions
diff --git a/libpyside/signalmanager.cpp b/libpyside/signalmanager.cpp index 7d10f77f4..3e5e3c07c 100644 --- a/libpyside/signalmanager.cpp +++ b/libpyside/signalmanager.cpp @@ -260,28 +260,38 @@ static bool emitShortCircuitSignal(QObject* source, int signalIndex, PyObject* a static bool emitNormalSignal(QObject* source, int signalIndex, const char* signal, PyObject* args, const QStringList& argTypes) { - int argsGiven = PySequence_Size(args); + Shiboken::AutoDecRef sequence(PySequence_Fast(args, 0)); + int argsGiven = PySequence_Fast_GET_SIZE(sequence.object()); if (argsGiven > argTypes.count()) { - QString msg = QString("%1 only accepts %2 arguments, %3 given!").arg(signal).arg(argTypes.count()).arg(argsGiven); - PyErr_SetString(PyExc_TypeError, msg.toLocal8Bit().constData()); + PyErr_Format(PyExc_TypeError, "%s only accepts %d arguments, %d given!", signal, argTypes.count(), argsGiven); return false; } void** signalArgs = new void*[argsGiven+1]; signalArgs[0] = 0; - for (int i = 0; i < argsGiven; ++i) - signalArgs[i+1] = Shiboken::TypeResolver::get(qPrintable(argTypes[i]))->toCpp(PySequence_GetItem(args, i)); + int i; + for (i = 0; i < argsGiven; ++i) { + Shiboken::TypeResolver* typeResolver = Shiboken::TypeResolver::get(qPrintable(argTypes[i])); + if (typeResolver) { + signalArgs[i+1] = typeResolver->toCpp(PySequence_Fast_GET_ITEM(sequence.object(), i)); + } else { + PyErr_Format(PyExc_TypeError, "Unknown type used to emit a signal: %s", qPrintable(argTypes[i])); + break; + } + } - QMetaObject::activate(source, signalIndex, signalArgs); + bool ok = i == argsGiven; + if (ok) + QMetaObject::activate(source, signalIndex, signalArgs); // FIXME: This will cause troubles with non-direct connections. - for (int i = 0; i < argsGiven; ++i) - Shiboken::TypeResolver::get(qPrintable(argTypes[i]))->deleteObject(signalArgs[i+1]); + for (int j = 0; j < i; ++j) + Shiboken::TypeResolver::get(qPrintable(argTypes[j]))->deleteObject(signalArgs[j+1]); delete[] signalArgs; - return true; + return ok; } bool SignalManager::emitSignal(QObject* source, const char* signal, PyObject* args) |