Struggling with signal which has a default parameter.

[JPNaude]

Ok, I've made the slot public and rebuilt. Still I get a null everytime.

From a brief look I see you are emitting layoutChanged() from your slot, and that one is with a default value, which would explain why in the debugger you see first the one with a non NULL object, and then one with NULL.
You have a recursion there.

I did mention it but probably not clear enough, the signal and slot are on different classes. Thus the layoutChanged() I emit in the slot is not the same, it is layoutChanged() on QAbstractItemModel.

[high_flyer]

the signal and slot are on different classes.

And the classes are also not related (through derivation) (just making sure)

[JPNaude]

Yes they are not related. I've checked that it only breaks once in that function as well.

As I said in my first post, it looks like the wrong signal is emitted by the meta object system.

int Qtilities::Core::Observer::qt_metacall(QMetaObject::Call _c, int _id, void **_a)
{
    _id = QObject::qt_metacall(_c, _id, _a);
    if (_id < 0)
        return _id;
    if (_c == QMetaObject::InvokeMetaMethod) {
        switch (_id) {
        case 0: modificationStateChanged((*reinterpret_cast< bool(*)>(_a[1]))); break;
        case 1: monitoredPropertyChanged((*reinterpret_cast< const char*(*)>(_a[1])),(*reinterpret_cast< QList<QObject*>(*)>(_a[2]))); break;
        case 2: monitoredPropertyChanged((*reinterpret_cast< const char*(*)>(_a[1]))); break;
        case 3: propertyChangeFiltered((*reinterpret_cast< const char*(*)>(_a[1])),(*reinterpret_cast< QList<QObject*>(*)>(_a[2]))); break;
        case 4: propertyChangeFiltered((*reinterpret_cast< const char*(*)>(_a[1]))); break;
        case 5: numberOfSubjectsChanged((*reinterpret_cast< Observer::SubjectChangeIndication(*)>(_a[1])),(*reinterpret_cast< QList<QObject*>(*)>(_a[2]))); break;
        case 6: numberOfSubjectsChanged((*reinterpret_cast< Observer::SubjectChangeIndication(*)>(_a[1]))); break;
        case 7: layoutChanged((*reinterpret_cast< QObject*(*)>(_a[1]))); break;
        case 8: layoutChanged(); break;
        default: ;
        }

int Qtilities::Core::Observer::qt_metacall(QMetaObject::Call _c, int _id, void **_a)
{
    _id = QObject::qt_metacall(_c, _id, _a);
    if (_id < 0)
        return _id;
    if (_c == QMetaObject::InvokeMetaMethod) {
        switch (_id) {
        case 0: modificationStateChanged((*reinterpret_cast< bool(*)>(_a[1]))); break;
        case 1: monitoredPropertyChanged((*reinterpret_cast< const char*(*)>(_a[1])),(*reinterpret_cast< QList<QObject*>(*)>(_a[2]))); break;
        case 2: monitoredPropertyChanged((*reinterpret_cast< const char*(*)>(_a[1]))); break;
        case 3: propertyChangeFiltered((*reinterpret_cast< const char*(*)>(_a[1])),(*reinterpret_cast< QList<QObject*>(*)>(_a[2]))); break;
        case 4: propertyChangeFiltered((*reinterpret_cast< const char*(*)>(_a[1]))); break;
        case 5: numberOfSubjectsChanged((*reinterpret_cast< Observer::SubjectChangeIndication(*)>(_a[1])),(*reinterpret_cast< QList<QObject*>(*)>(_a[2]))); break;
        case 6: numberOfSubjectsChanged((*reinterpret_cast< Observer::SubjectChangeIndication(*)>(_a[1]))); break;
        case 7: layoutChanged((*reinterpret_cast< QObject*(*)>(_a[1]))); break;
        case 8: layoutChanged(); break;
        default: ;
        }

To copy to clipboard, switch view to plain text mode 

As I said, the debugger steps into case 8 in the above code so I'm pretty sure the problem is with the delivery of the signal, rather than the receiving side. At least it looks like that.

One thing I don't understand is the following:
The class emitting the signal is called Observer. The instance on which the signal is emitted is called TreeNode which inherits Observer. Stepping through the debugger the signal emission goes into the qt_metacall function of TreeNode and then basically calls qt_metacall on the Observer base class. This is the piece of code I'm referring to:

int Qtilities::CoreGui::TreeNode::qt_metacall(QMetaObject::Call _c, int _id, void **_a)
{
    _id = Observer::qt_metacall(_c, _id, _a);
    if (_id < 0)
        return _id;

int Qtilities::CoreGui::TreeNode::qt_metacall(QMetaObject::Call _c, int _id, void **_a)
{
    _id = Observer::qt_metacall(_c, _id, _a);
    if (_id < 0)
        return _id;

To copy to clipboard, switch view to plain text mode 

The above funciton is entered with _id = 12, however as soon as it steps into the base class observer the base class gets _id = 8 as the parameter which is the signal with the default argument. Maybe I don't know the moc system that well and its correct but it does not make sense to me.

[high_flyer]

The class emitting the signal is called Observer. The instance on which the signal is emitted is called TreeNode which inherits Observer.

I don't understand this - the instance should BE Objserver instance, not inherit it - which would mean its an instance of a class that inherits Observer - can you clarify this - since if this is confused, it can well be the root of the problem!

Lets try to do this in order:
What object (which class is it, what is its instance (variable/obejct) name?
What object declares and implements the slot?

based on your original post:

connect(d_observer,SIGNAL(layoutChanged(QObject*)),SLOT(rebuildTreeStructure(QObject*)));

connect(d_observer,SIGNAL(layoutChanged(QObject*)),SLOT(rebuildTreeStructure(QObject*)));

To copy to clipboard, switch view to plain text mode 

The sender class and instance is also the slot class and instance, and in the slot you are emitting the same signal again, even though you said its another class...

[wysota]

Change the name of the signal and the slot to something not clashing with existing signals in Qt. You can also try declaring two separate slots - one with the QObject param and the other without it and see if it changes anything. It could be that moc has a bug somewhere that makes it work incorrectly with slots that have default params.

[JPNaude]

Ok no problem let me explain in more detail:

// This is a simplified view of the observer class which contains the signal. The signal is also emitted in one of this class's functions.
class Observer : public QObject
{
    Q_OBJECT
 
public:
    Observer();
    virtual ~Observer();
 
    void someFunctionEmittingTheSignal() {
        QObject* new_obj = new QObject();
        emit layoutChanged(new_obj);
    }
 
signals:
    void layoutChanged(QObject* added_object = 0);
 
}
 
// This is the class of which the instance is created:
class TreeNode: public Observer
{
    Q_OBJECT
 
public:
    TreeNode();
    virtual ~TreeNode();
}

// This is a simplified view of the observer class which contains the signal. The signal is also emitted in one of this class's functions.
class Observer : public QObject
{
    Q_OBJECT

public:
    Observer();
    virtual ~Observer();

    void someFunctionEmittingTheSignal() {
        QObject* new_obj = new QObject();
        emit layoutChanged(new_obj);
    }

signals:
    void layoutChanged(QObject* added_object = 0);

}

// This is the class of which the instance is created:
class TreeNode: public Observer
{
    Q_OBJECT

public:
    TreeNode();
    virtual ~TreeNode();
}

To copy to clipboard, switch view to plain text mode 

I have another class with the slot (again simplified):

// This is the class of which the instance is created:
class ObserverTreeModel: public QAbstractItemModel
{
    Q_OBJECT
 
public:
    ObserverTreeModel();
    virtual ~ObserverTreeModel();
    setObserverContext(Observer* obs) {
        d_observer = obs;
        connect(d_observer,SIGNAL(layoutChanged(QObject*)),SLOT(rebuildTreeStructure(QObject*)));
    }
 
private slots:
    void rebuildTreeStructure(Object* obj = 0) {
        // Do some stuff that is going to change the layout of the tree.
        emit layoutAboutToChange(); // Emits QAbstractItemModel::layoutAboutToChange();
        emit layoutChanged(); // Emits QAbstractItemModel::layoutChanged();
    }
 
private:
    Observer* d_observer;
}

// This is the class of which the instance is created:
class ObserverTreeModel: public QAbstractItemModel
{
    Q_OBJECT

public:
    ObserverTreeModel();
    virtual ~ObserverTreeModel();
    setObserverContext(Observer* obs) {
        d_observer = obs;
        connect(d_observer,SIGNAL(layoutChanged(QObject*)),SLOT(rebuildTreeStructure(QObject*)));
    }

private slots:
    void rebuildTreeStructure(Object* obj = 0) {
        // Do some stuff that is going to change the layout of the tree.
        emit layoutAboutToChange(); // Emits QAbstractItemModel::layoutAboutToChange();
        emit layoutChanged(); // Emits QAbstractItemModel::layoutChanged();
    }

private:
    Observer* d_observer;
}

To copy to clipboard, switch view to plain text mode 

I do something like this then:

ObserverTreeModel model;
TreeNode tree_node;
model.setObserverContext(&tree_node);
tree_node.someFunctionEmittingTheSignal();

ObserverTreeModel model;
TreeNode tree_node;
model.setObserverContext(&tree_node);
tree_node.someFunctionEmittingTheSignal();

To copy to clipboard, switch view to plain text mode 

I hope that explains it better.

You can also try declaring two separate slots - one with the QObject param and the other without it and see if it changes anything. It could be that moc has a bug somewhere that makes it work incorrectly with slots that have default params.

I've removed the default parameter on the slots side, but it did not make a difference.

[wysota]

So the only explanation I can see is that your parameter has been deleted somewhere in the meantime. By the way, is line #15 of your last snippet a typo or a real code?

[JPNaude]

So the only explanation I can see is that your parameter has been deleted somewhere in the meantime. By the way, is line #15 of your last snippet a typo or a real code?

I'm very sure that its not deleted because I can access it in the tree view directly after this function. Also if it was a QPointer and it was deleted it would make sense but if it was deleted in my case I should get an invalid pointer.

Yes that was a typo, thanks.

[wysota]

The problem is what you said in post #13 - QMetaObject is stepping into a wrong case. It should go into 7 and not 8. Can you show us all places where you emit this signal? In post #6 there is an emit layoutChanged() but it seems this is a different "layoutChanged()".

[JPNaude]

I've updated my example in post #16 to show what the other layoutChanged() signal is. It will make more sense now, they should be independent. I can try to change the signal names as you said to see if it makes any difference.

[JPNaude]

I figured it out...

Was my mistake. The observer class I mentioned is able to observe other observers creating a tree structure. The problem was that I emitted the signal in an observer low down in the tree and as the signal propagated through to the top observer (connected to the view) the parameter got lost since those connections were made without the parameters. I forgot that this is happening and thought I emitted the signal on the top level observer. My bad.

Thanks for your help and time again.
Cheers,
Jaco