#include <QTime>
#include <QThread>
#include <QCoreApplication>
-
+#include <QDebug>
#define USE_QPROCESS
// Reuse of instance
#ifndef USE_QPROCESS
-int Systemcall::startscript(Starttype how, string const & what)
+int Systemcall::startscript(Starttype how, string const & what,
+ bool /*process_events*/)
{
string command = what;
-int Systemcall::startscript(Starttype how, string const & what)
+int Systemcall::startscript(Starttype how, string const & what, bool process_events)
{
string outfile;
QString cmd = toqstr(parsecmd(what, outfile));
SystemcallPrivate d(outfile);
- bool processEvents = false;
d.startProcess(cmd);
- if (!d.waitWhile(SystemcallPrivate::Starting, processEvents, 3000)) {
- LYXERR0("QProcess " << cmd << " did not start!");
+ if (!d.waitWhile(SystemcallPrivate::Starting, process_events, -1)) {
+ LYXERR0("Systemcall: '" << cmd << " did not start!");
LYXERR0("error " << d.errorMessage());
return 10;
}
if (how == DontWait) {
- // TODO delete process later
+ QProcess* released = d.releaseProcess();
+ (void) released; // TODO who deletes it?
return 0;
}
- if (!d.waitWhile(SystemcallPrivate::Running, processEvents, 180000)) {
- LYXERR0("QProcess " << cmd << " did not finished!");
+ if (!d.waitWhile(SystemcallPrivate::Running, process_events, 180000)) {
+ LYXERR0("Systemcall: '" << cmd << "' did not finished!");
LYXERR0("error " << d.errorMessage());
LYXERR0("status " << d.exitStatusMessage());
return 20;
int const exit_code = d.exitCode();
if (exit_code) {
- LYXERR0("QProcess " << cmd << " finished!");
- LYXERR0("error " << exit_code << ": " << d.errorMessage());
+ LYXERR0("Systemcall: '" << cmd << "' finished with exit code " << exit_code);
}
- d.flush();
- d.killProcess();
-
return exit_code;
}
SystemcallPrivate::SystemcallPrivate(const std::string& of) :
- proc_(new QProcess), outindex_(0),
- errindex_(0), showout_(false), showerr_(false), outfile(of)
+ proc_(new QProcess), outindex_(0), errindex_(0),
+ outfile(of), showout_(false), showerr_(false), process_events(false)
{
if (!outfile.empty()) {
// Check whether we have to simply throw away the output.
void SystemcallPrivate::startProcess(const QString& cmd)
{
- state = SystemcallPrivate::Starting;
- proc_->start(cmd);
+ if (proc_) {
+ state = SystemcallPrivate::Starting;
+ proc_->start(cmd);
+ }
+}
+
+
+void SystemcallPrivate::processEvents()
+{
+ if(process_events) {
+ //static int count = 0; qDebug() << count++ << ": waitAndProcessEvents";
+ QCoreApplication::processEvents(QEventLoop::AllEvents);
+ }
}
void SystemcallPrivate::waitAndProcessEvents()
{
Sleep::millisec(100);
- QCoreApplication::processEvents(QEventLoop::ExcludeUserInputEvents);
+ processEvents();
}
-bool SystemcallPrivate::waitWhile(State waitwhile, bool processEvents, int timeout)
+bool SystemcallPrivate::waitWhile(State waitwhile, bool proc_events, int timeout)
{
+ if (!proc_)
+ return false;
+
+ process_events = proc_events;
+
// Block GUI while waiting,
// relay on QProcess' wait functions
- if (!processEvents) {
+ if (!process_events) {
if (waitwhile == Starting)
return proc_->waitForStarted(timeout);
if (waitwhile == Running)
}
-
SystemcallPrivate::~SystemcallPrivate()
{
+ flush();
+
if (outindex_) {
outdata_[outindex_] = '\0';
outindex_ = 0;
cerr << errdata_;
}
cerr.flush();
+
+ killProcess();
}
void SystemcallPrivate::flush()
{
- // If the output has been redirected, we write it all at once.
- // Even if we are not running in a terminal, the output could go
- // to some log file, for example ~/.xsession-errors on *nix.
- if (!os::is_terminal(os::STDOUT) && outfile.empty())
- cout << fromqstr(QString::fromLocal8Bit(
- proc_->readAllStandardOutput().data()));
- if (!os::is_terminal(os::STDERR))
- cerr << fromqstr(QString::fromLocal8Bit(
- proc_->readAllStandardError().data()));
+ if (proc_) {
+ // If the output has been redirected, we write it all at once.
+ // Even if we are not running in a terminal, the output could go
+ // to some log file, for example ~/.xsession-errors on *nix.
+ if (!os::is_terminal(os::STDOUT) && outfile.empty())
+ cout << fromqstr(QString::fromLocal8Bit(
+ proc_->readAllStandardOutput().data()));
+ if (!os::is_terminal(os::STDERR))
+ cerr << fromqstr(QString::fromLocal8Bit(
+ proc_->readAllStandardError().data()));
+ }
}
+
void SystemcallPrivate::stdOut()
{
- if (showout_) {
+ if (proc_ && showout_) {
char c;
proc_->setReadChannel(QProcess::StandardOutput);
while (proc_->getChar(&c)) {
}
}
}
+ processEvents();
}
void SystemcallPrivate::stdErr()
{
- if (showerr_) {
+ if (proc_ && showerr_) {
char c;
proc_->setReadChannel(QProcess::StandardError);
while (proc_->getChar(&c)) {
}
}
}
+ processEvents();
+}
+
+
+void SystemcallPrivate::processStarted()
+{
+ state = Running;
+ // why do we get two started signals?
+ //disconnect(proc_, SIGNAL(started()), this, SLOT(processStarted()));
+}
+
+
+void SystemcallPrivate::processFinished(int, QProcess::ExitStatus)
+{
+ state = Finished;
}
-void SystemcallPrivate::processError(QProcess::ProcessError err)
+void SystemcallPrivate::processError(QProcess::ProcessError)
{
state = Error;
}
QString SystemcallPrivate::errorMessage() const
{
+ if (!proc_)
+ return "No QProcess available";
+
QString message;
switch (proc_->error()) {
case QProcess::FailedToStart:
}
-void SystemcallPrivate::processStarted()
-{
- state = Running;
- // why do we get two started signals?
- //disconnect(proc_, SIGNAL(started()), this, SLOT(processStarted()));
-}
-
-
-void SystemcallPrivate::processFinished(int, QProcess::ExitStatus status)
-{
- state = Finished;
-}
-
-
QString SystemcallPrivate::exitStatusMessage() const
{
+ if (!proc_)
+ return "No QProcess available";
+
QString message;
switch (proc_->exitStatus()) {
case QProcess::NormalExit:
return message;
}
+
int SystemcallPrivate::exitCode()
{
+ if (!proc_)
+ return -1;
+
return proc_->exitCode();
}
+QProcess* SystemcallPrivate::releaseProcess()
+{
+ QProcess* released = proc_;
+ proc_ = 0;
+ return released;
+}
+
+
void SystemcallPrivate::killProcess()
{
killProcess(proc_);
}
+
void SystemcallPrivate::killProcess(QProcess * p)
{
- p->disconnect();
- p->closeReadChannel(QProcess::StandardOutput);
- p->closeReadChannel(QProcess::StandardError);
- p->close();
- delete p;
+ if (p) {
+ p->disconnect();
+ p->closeReadChannel(QProcess::StandardOutput);
+ p->closeReadChannel(QProcess::StandardError);
+ p->close();
+ delete p;
+ }
}