QObject::killTimers() warning after timer stopped from its own thread

[astodolski]

ok, next try. Does this give same thread timer warning?

FlashWizard::FlashWizard(...)
{
...
  worker = new Worker;
 
  qDebug() << worker->thread();
 
  workerThread = new QThread;
  worker->moveToThread(workerThread);
 
  qDebug() << worker->thread();
  qDebug() << worker->m_workerTimer->thread();
 
  //connect(workerThread, SIGNAL(started()), worker, SLOT(start()));
    connect(workerThread, SIGNAL(finished()), worker, SLOT(stop()));
    connect(worker, SIGNAL(finished()), workerThread, SLOT(quit()));
    connect(worker, SIGNAL(finished()), ui->programPage, SLOT(setDone()));
    connect(worker, SIGNAL(updateProgress(int)), this, SLOT(updateWritingProgressBar(int)));
 
  workerThread->start();
  QMetaObject::invokeMethod(worker, "start");
  QMetaObject::invokeMethod(worker, "stop");
}

FlashWizard::FlashWizard(...)
{
...
  worker = new Worker;

  qDebug() << worker->thread();

  workerThread = new QThread;
  worker->moveToThread(workerThread);

  qDebug() << worker->thread();
  qDebug() << worker->m_workerTimer->thread();

  //connect(workerThread, SIGNAL(started()), worker, SLOT(start()));
    connect(workerThread, SIGNAL(finished()), worker, SLOT(stop()));
    connect(worker, SIGNAL(finished()), workerThread, SLOT(quit()));
    connect(worker, SIGNAL(finished()), ui->programPage, SLOT(setDone()));
    connect(worker, SIGNAL(updateProgress(int)), this, SLOT(updateWritingProgressBar(int)));

  workerThread->start();
  QMetaObject::invokeMethod(worker, "start");
  QMetaObject::invokeMethod(worker, "stop");
}

To copy to clipboard, switch view to plain text mode 

Yes the same message. For what it's worth, I want to note that I get the warning both calling start in the constructor or calling start (where it originally resides) in the slot that's called for the page that displays the progress bar. I just wanted to see if there was a difference - there isn't.

I took out calling new on the timer in the constructor for the worker. So I now use a reference for the timer in the connect call in the rather than a pointer in start method in the worker:

void Worker::start()
{
    connect(&m_workerTimer, SIGNAL(timeout()), this, SLOT(doWork()));
    m_workerTimer.start(100);
    doWork();
}

void Worker::start()
{
    connect(&m_workerTimer, SIGNAL(timeout()), this, SLOT(doWork()));
    m_workerTimer.start(100);
    doWork();
}

To copy to clipboard, switch view to plain text mode 

Unexpectedly, I don't see the warning now until the worker completes - not when I click the cancel button, which was the previous case.

void FlashWizard::reject()
{
    if (workerThread->isRunning())
    {
        worker->stop();
    }
}

void FlashWizard::reject()
{
    if (workerThread->isRunning())
    {
        worker->stop();
    }
}

To copy to clipboard, switch view to plain text mode