unable to terminate QProcess in QThread, returns wrong status

[jacek]

Qt docs say:

int QProcess::execute ( const QString & program, const QStringList & arguments ) [static]
Starts the program program with the arguments arguments in a new process, waits for it to finish, and then returns the exit code of the process. Any data the new process writes to the console is forwarded to the calling process.
The environment and working directory are inherited by the calling process.
On Windows, arguments that contain spaces are wrapped in quotes.
int QProcess::execute ( const QString & program ) [static]
This is an overloaded member function, provided for convenience.
Starts the program program in a new process. program is a single string of text containing both the program name and its arguments. The arguments are separated by one or more spaces.

These are static methods, so you don't need QProcess instance to invoke them and even if you do use it, they won't influence it.

[jpn]

Are you sure you even need a thread..? QProcess launches an external process so it won't block anything.

Edit: Oh, and by the way, you are calling myprocess object's methods from 2 separate threads.

QMyThread::teminateProcess() and QMyThread::stateOfProcess() are executed in the main thread. QMyThread::run() is executed in the separate thread.

[sector]

Ah,.. you are right but if I make something like this

void myQtApp::launch()
{
	QProcess proc;
 
	proc.start("regedit.exe");
 
}

void myQtApp::launch()
{
	QProcess proc;

	proc.start("regedit.exe");

}

To copy to clipboard, switch view to plain text mode 

I get QProcess: Destroyed while process is still running.

If I define proc as member of myQtApp it works, nah.. :] This tempted me to use execute() and thread.. It works fine, thanks for your time.

Edit: Oh, and by the way, you are calling myprocess object's methods from 2 separate threads.

QMyThread::teminateProcess() and QMyThread::stateOfProcess() are executed in the main thread. QMyThread::run() is executed in the separate thread.

Is there something wrong with that ?

[jacek]

I get QProcess: Destroyed while process is still running.

Because proc gets destroyed as soon as launch() returns. You can create it on the heap (i.e. using new operator) to avoid this.

Is there something wrong with that ?

QProcess, just as every class derived from QObject, isn't thread-safe.