Issue emitting signal from background thread.

[Chris@Link]

void MainWindow::startup()
      // Get pointer to layout manager for main window
      QLayout* layout = this->layout();
      // Add LCD screen to the main window layout at specified position
      LCD = new(IPR5000_LCD)(this);
      LCD->setGeometry(430,50,320,240);
      LCD->setParent(this);
      layout->addWidget(LCD);
 
      KeyManagementAgent key_management_agent;
      SettingsFileRetrievalAgent settings_agent( key_management_agent );
      connect(&settings_agent, SIGNAL(updateSettingsNotification()), LCD, SLOT(restarting()));
      // move it to its own thread and do stuff with it.
     QThread *settingsFileUpdaterThread = new QThread;
     settings_agent.moveToThread(settingsFileUpdaterThread);
     settingsFileUpdaterThread->start();
     QMetaObject::invokeMethod(&settings_agent, "worker_thread", Qt::QueuedConnection);
 
}

void MainWindow::startup()
      // Get pointer to layout manager for main window
      QLayout* layout = this->layout();
      // Add LCD screen to the main window layout at specified position
      LCD = new(IPR5000_LCD)(this);
      LCD->setGeometry(430,50,320,240);
      LCD->setParent(this);
      layout->addWidget(LCD);

      KeyManagementAgent key_management_agent;
      SettingsFileRetrievalAgent settings_agent( key_management_agent );
      connect(&settings_agent, SIGNAL(updateSettingsNotification()), LCD, SLOT(restarting()));
      // move it to its own thread and do stuff with it.
     QThread *settingsFileUpdaterThread = new QThread;
     settings_agent.moveToThread(settingsFileUpdaterThread);
     settingsFileUpdaterThread->start();
     QMetaObject::invokeMethod(&settings_agent, "worker_thread", Qt::QueuedConnection);

}

To copy to clipboard, switch view to plain text mode 

void IPR5000_LCD::restarting()
{
  ui->stackedWidget->setCurrentIndex(2);
}

void IPR5000_LCD::restarting()
{
  ui->stackedWidget->setCurrentIndex(2);
}

To copy to clipboard, switch view to plain text mode 

Like I said previously I do this sort of thing other places without problems so I don't see any reason there would be a problem with the 'LCD', it just happens to be this thread that is giving me issues.

Commenting out the code in the restarting() method still results in a segfault. I tried printing a simple debug message and it still crashes. I don't think it's a problem with the LCD class. The LCD slot is never called because the signal is never emitted, that's how I know it cant be the LCD.

Edit: It will loop about 5-8 times before it throws a segfault.

I really appreciate the help.

[high_flyer]

1. Is 'key_management_agent' a parent for 'settings_agent' or just a parameter?
2. settings_agent is a stack variable, which dies at the end of the constructor. I would expect moveToThread() to have a problem with that - try allocating it on the heap.
This would also explain this==NULL.

Like I said previously I do this sort of thing other places without problems so I don't see any reason there would be a problem with the 'LCD', it just happens to be this thread that is giving me issues.

1.LCD was a likely candidate - since I don't know your code I wanted to rule it out.
2.I fail to see the relevance of the claim that something worked somewhere else since obviously its not *exactly* the same, and secondly, obviously it doesn't work here - so it helps little that it does somewhere else.
Specially when threads are in question - since alone different hardware (same software) may produce different results, or bring bugs to the surface on one system that are not see on another.
For your own good - stop thinking "this has to work" if it doesn't - rather "I must be doing something wrong if it doesn't work".
By the way, the code as you posted it, really requires no threads.

[Chris@Link]

1. Is 'key_management_agent' a parent for 'settings_agent' or just a parameter?
2. settings_agent is a stack variable, which dies at the end of the constructor. I would expect moveToThread() to have a problem with that - try allocating it on the heap.
This would also explain this==NULL.

Upon further investigation the settings_agent and the key_management_agent was being destroyed after we finished the startup function. I don't know how I managed to overlook that but I did.

2.I fail to see the relevance of the claim that something worked somewhere else since obviously its not *exactly* the same, and secondly, obviously it doesn't work here - so it helps little that it does somewhere else.
Specially when threads are in question - since alone different hardware (same software) may produce different results, or bring bugs to the surface on one system that are not see on another.
For your own good - stop thinking "this has to work" if it doesn't - rather "I must be doing something wrong if it doesn't work".
By the way, the code as you posted it, really requires no threads.

I understand, I was extremely frustrated and I overlooked the fact that the stack variable was being destroyed.


Thanks for the help.