Deleting QProcess after finish

mirswith

excellent, this is exactly what I was looking for.

Thanks!

lgeyer

[quote author="mirswith" date="1316459702"]The signal queue is also what I am suspecting to be the cause, I suppose I could emit my own signal once waitForFinished() returns and delete the process there. This is of course assuming the signal queue is the culprit and that I would be gauranteed that my emit would occur after anything already in the queue.[/quote]

Why do you even connect to signals of QProcess in the first place?

They have no chance to be delivered, because your thread is suspended until waitForFinished() returns - thus no event processing occours. As soon as waitForFinished() returns it is absolutely safe to process to QProcess output and delete the QProcess (given that no signals have been connected).

You either use signals or you use waitForFinished(). There is no sense in using both.

mirswith

Actually I am running a number of QProcesses in a QThread which separates them from my GUI thread and I am receiving signals off of them. I am using the signals purely to capture standard/error out messages, however I have not tried to just read them after waitForFinished() which might work just fine.

mirswith

I should probably also mention I need the signals so that I can monitor the output as it progresses, just a dump at the end is not granular enough for my needs.

-=ben

goetz

QProcess, by it's nature, runs asynchronously by design. Why do you put that into a separate thread? Connecting to signals should be sufficient.

mirswith

Its the architecture of the tool I am developing. It's basically a system that queues a number of jobs, each job get's its own QThread to run in. A job then consists of one or more tasks defined with javascript, these scripts can then execute system tools using QProcess and I want these to be synchronous in their execution order.

-=ben

goetz

That sounds reasonable. But in that case, I would define my own signals and emit them after the process has finished and you have read stdout and/or stderr.

mirswith

If I wanted to implement my own waitForFinished() what would you recommend?

goetz

The usual way of defining a new signal. That completely depends on your use case and needs. The outline is basically:

@
class MyProcessWorker : public ...
{
// constructors etc.

signals:
myProcessFinished(const QString &message);

protected:
void runMyProcess();
};

void MyProcessWorker::runMyProcess()
{
QProcess proc;
proc.start( args... );
proc.waitForFinished(-1);

QByteArray stdout = proc.readAllStandardOutput();
QByteArray stderr = proc.readAllStandardError();

QString msg = whatever you make out of stdout and stderr;

emit myProcessFinished(msg);

}
@

mirswith

Thanks, I understand signals/slots I was asking because I was running into an issue where I wanted a different thread to be able to cancel the process and if I sit on QProcesses waitForFinished() and then call kill() from a different thread then nasty things happen. :)

What I did in the end was simply set a bool that determines if I should kill the process and then after I call the process I do:

@
while(proc->waitForFinished(250))
{
if( _killRequested ) proc->kill();
}
@

This seems to be working well.