Lost widget

[anli]

while(eThread.isRunning())
        usleep(100);

while(eThread.isRunning())
        usleep(100);

To copy to clipboard, switch view to plain text mode 

But what is this for? It stops the main thread. And if so, why do you need another thread? You can move its functionality to the main thread and the result will be identical. Better yet, you can split the work you do in the other thread into smaller chunks and use a QTimer with 0 interval to trigger next steps of the process and you'll gain this, that you won't block the event handling and your gui will remain responsive.


"Common" doesn't mean "proper". You can achieve the same without using a separate thread which only slows down the process.

I have rewrote the code. My previous msg is in accordans with these fragments:

Starting a thread:

ExcitThread* eThread = new ExcitThread(this, eInfo, this);
    eThread->start(QThread::LowestPriority);

ExcitThread* eThread = new ExcitThread(this, eInfo, this);
    eThread->start(QThread::LowestPriority);

To copy to clipboard, switch view to plain text mode 

QThread implementation:

ExcitThread::ExcitThread(QObject* parent, const ExcitInfo& anInfo, QWidget* aFeedback) : QThread(parent) {
    this->info = anInfo;
    this->feedback = aFeedback;
    connect(this, SIGNAL(showStatus(const QString&)), this->feedback, SLOT(showStatus(const QString&)));
    connect(this, SIGNAL(showStatus(const QString&, int)), this->feedback, SLOT(showStatus(const QString&, int)));
    connect(this, SIGNAL(showCritical(const QString&)), this->feedback, SLOT(showCritical(const QString&)));
}
 
void ExcitThread::run() {
    emit showStatus("Generating excitation...");
    try {
        Excitation::generate( this->info.workDir.toStdString(),
                              this->info.duration,
                              this->info.sampleRate,
                              this->info.bitDepth,
                              this->info.minFreq,
                              this->info.maxFreq
                            );
    } catch(QLE e) {
        emit showCritical(e.msg.c_str());
        emit showStatus("Excitation generating failed!", 2000);
        return;
    }
    emit showStatus("Excitation and inverse filter are generated!", 2000);
}

ExcitThread::ExcitThread(QObject* parent, const ExcitInfo& anInfo, QWidget* aFeedback) : QThread(parent) {
    this->info = anInfo;
    this->feedback = aFeedback;
    connect(this, SIGNAL(showStatus(const QString&)), this->feedback, SLOT(showStatus(const QString&)));
    connect(this, SIGNAL(showStatus(const QString&, int)), this->feedback, SLOT(showStatus(const QString&, int)));
    connect(this, SIGNAL(showCritical(const QString&)), this->feedback, SLOT(showCritical(const QString&)));
}

void ExcitThread::run() {
    emit showStatus("Generating excitation...");
    try {
        Excitation::generate( this->info.workDir.toStdString(),
                              this->info.duration,
                              this->info.sampleRate,
                              this->info.bitDepth,
                              this->info.minFreq,
                              this->info.maxFreq
                            );
    } catch(QLE e) {
        emit showCritical(e.msg.c_str());
        emit showStatus("Excitation generating failed!", 2000);
        return;
    }
    emit showStatus("Excitation and inverse filter are generated!", 2000);
}

To copy to clipboard, switch view to plain text mode 

That "while" was with the aim to understand how all this works :-)