Register Signal objects on MetaObject.

Reviewer: Hugo Parente Lima <hugo.lima@openbossa.org>,
          Luciano Wolf <luciano.wolf@openbossa.org>

libpyside/dynamicqmetaobject.cpp

@@ -61,7 +61,7 @@ static int registerString(const QByteArray& s, QList<QByteArray>* strings)
     return idx;
 }
 
-MethodData::MethodData(const char *signature, const char *type)
+MethodData::MethodData(const char* signature, const char* type)
     : m_signature(signature), m_type(type)
 {
 }
@@ -72,11 +72,16 @@ void MethodData::clear()
     m_type.clear();
 }
 
-bool MethodData::operator==(const MethodData &other) const
+bool MethodData::operator==(const MethodData& other) const
 {
     return m_signature == other.signature();
 }
 
+bool MethodData::operator==(const char* other) const
+{
+    return m_signature == other;
+}
+
 QByteArray MethodData::signature() const
 {
     return m_signature;
@@ -107,9 +112,13 @@ DynamicQMetaObject::~DynamicQMetaObject()
 
 void DynamicQMetaObject::addSignal(const char* signal, const char* type)
 {
+    QLinkedList<MethodData>::iterator i = qFind(m_signals.begin(), m_signals.end(), signal);
+    if (i != m_signals.end())
+        return;
+
     //search for a empty space
     MethodData blank;
-    QLinkedList<MethodData>::iterator i = qFind(m_signals.begin(), m_signals.end(), blank);
+    i = qFind(m_signals.begin(), m_signals.end(), blank);
     if (i != m_signals.end()) {
         *i = MethodData(signal, type);
         updateMetaObject();
@@ -127,9 +136,13 @@ void DynamicQMetaObject::addSignal(const char* signal, const char* type)
 
 void DynamicQMetaObject::addSlot(const char* slot, const char* type)
 {
+    QLinkedList<MethodData>::iterator i = qFind(m_slots.begin(), m_slots.end(), slot);
+    if (i != m_slots.end())
+        return;
+
     //search for a empty space
     MethodData blank;
-    QLinkedList<MethodData>::iterator i = qFind(m_slots.begin(), m_slots.end(), blank);
+    i = qFind(m_slots.begin(), m_slots.end(), blank);
     if (i != m_slots.end()) {
         *i = MethodData(slot, type);
     } else {
@@ -162,18 +175,15 @@ DynamicQMetaObject* DynamicQMetaObject::createBasedOn(PyObject* pyObj, PyTypeObj
 
     while (PyDict_Next(type->tp_dict, &pos, &key, &value)) {
 
-#if 0
         //Register signals
-        if (value->ob_type == &PySideSignal_Type) {
+        if (value->ob_type == &Signal_Type) {
-            SignalData *data = reinterpret_cast<SignalData*>(value);
+            PyObject *attr = PyObject_GetAttr(pyObj, key);
-
+            SignalInstanceData *data = reinterpret_cast<SignalInstanceData*>(attr);
-            for(int i=0, i_max=data->signatures_size; i < i_max; i++) {
+            while(data) {
-                mo->addSignal(data->signatures[i]);
+                mo->addSignal(data->signature);
-                printf("QObject with signal:[%s][%s]\n",
+                data = reinterpret_cast<SignalInstanceData*>(data->next);
-                       PyString_AS_STRING(PyObject_Str(key)), data->signatures[i]);
             }
         }
-#endif
 
         if (!PyFunction_Check(value))
             continue;

libpyside/dynamicqmetaobject.h

@@ -52,11 +52,12 @@ class MethodData
 {
 public:
     MethodData(){}
-    MethodData(const char *signature, const char *type);
+    MethodData(const char* signature, const char* type);
     void clear();
     QByteArray signature() const;
     QByteArray type() const;
-    bool operator==(const MethodData &other) const;
+    bool operator==(const MethodData& other) const;
+    bool operator==(const char* other) const;
 
 private:
     QByteArray m_signature;

libpyside/pyside.h

@@ -41,7 +41,7 @@
 namespace PySide
 {
 
-PYSIDE_API  void init(PyObject *module);
+PYSIDE_API void init(PyObject *module);
 
 } //namespace PySide
 

libpyside/qsignal.cpp

@@ -190,7 +190,7 @@ PyObject* signal_instance_get_item(PyObject* self, PyObject* key)
     return 0;
 }
 
-void signal_update_source(PyObject* source)
+void signalUpdateSource(PyObject* source)
 {
     PyObject* attrs = PyObject_Dir(source);
     for(int i = 0, i_max = PyList_Size(attrs); i < i_max; i++) {
@@ -330,6 +330,7 @@ void signal_instance_free(void* self)
 void signal_instance_initialize(PyObject* instance, PyObject* name, SignalData* data, PyObject* source, int index)
 {
     SignalInstanceData *self = reinterpret_cast<SignalInstanceData*>(instance);
+    self->next = 0;
     if (data->signalName)
         self->signalName = strdup(data->signalName);
     else
@@ -452,7 +453,7 @@ PyObject* signal_instance_emit(PyObject* self, PyObject* args)
     return PyObject_CallObject(pyMethod, tupleArgs);
 }
 
-PyObject* signal_new(const char* name, ...)
+PyObject* signalNew(const char* name, ...)
 {
     va_list listSignatures;
     char* sig;

libpyside/qsignal.h

@@ -21,10 +21,10 @@ extern "C"
 {
     PyAPI_DATA(PyTypeObject) Signal_Type;
     PyAPI_DATA(PyTypeObject) SignalInstance_Type;
-    PYSIDE_API PyAPI_FUNC(PyObject*) signal_new(const char* name, ...);
 }; //extern "C"
 
-PYSIDE_API void signal_update_source(PyObject* source);
+PYSIDE_API PyAPI_FUNC(PyObject*) signalNew(const char* name, ...);
+PYSIDE_API void signalUpdateSource(PyObject* source);
 
 } //namespace PySide