forked from qt-creator/qt-creator
Use QtcProcess for reading version number in uncrustify
Change-Id: I0c39342c692d1b4a5dfcf8fa562f5f300f6fc295 Reviewed-by: Artem Sokolovskii <artem.sokolovskii@qt.io> Reviewed-by: hjk <hjk@qt.io>
This commit is contained in:
@@ -54,7 +54,7 @@ const char SETTINGS_NAME[] = "uncrustify";
|
||||
UncrustifySettings::UncrustifySettings() :
|
||||
AbstractSettings(SETTINGS_NAME, ".cfg")
|
||||
{
|
||||
connect(&m_versionProcess, QOverload<int, QProcess::ExitStatus>::of(&QProcess::finished),
|
||||
connect(&m_versionProcess, &Utils::QtcProcess::finished,
|
||||
this, &UncrustifySettings::parseVersionProcessResult);
|
||||
|
||||
setCommand("uncrustify");
|
||||
@@ -229,13 +229,13 @@ void UncrustifySettings::updateVersion()
|
||||
m_versionProcess.kill();
|
||||
m_versionProcess.waitForFinished();
|
||||
}
|
||||
m_versionProcess.start(command().toString(), {"--version"});
|
||||
m_versionProcess.setCommand({ command(), { "--version" } });
|
||||
m_versionProcess.start();
|
||||
}
|
||||
|
||||
void UncrustifySettings::parseVersionProcessResult(int exitCode, QProcess::ExitStatus exitStatus)
|
||||
void UncrustifySettings::parseVersionProcessResult()
|
||||
{
|
||||
Q_UNUSED(exitCode)
|
||||
if (exitStatus != QProcess::NormalExit)
|
||||
if (m_versionProcess.exitStatus() != QProcess::NormalExit)
|
||||
return;
|
||||
|
||||
if (!parseVersion(QString::fromUtf8(m_versionProcess.readAllStandardOutput()), m_version))
|
||||
|
@@ -27,7 +27,7 @@
|
||||
|
||||
#include "../abstractsettings.h"
|
||||
#include <utils/fileutils.h>
|
||||
#include <QProcess>
|
||||
#include <utils/qtcprocess.h>
|
||||
|
||||
namespace Beautifier {
|
||||
namespace Internal {
|
||||
@@ -66,8 +66,8 @@ public:
|
||||
void setUseSpecificConfigFile(bool useConfigFile);
|
||||
|
||||
private:
|
||||
QProcess m_versionProcess;
|
||||
void parseVersionProcessResult(int exitCode, QProcess::ExitStatus exitStatus);
|
||||
Utils::QtcProcess m_versionProcess;
|
||||
void parseVersionProcessResult();
|
||||
};
|
||||
|
||||
} // namespace Internal
|
||||
|
Reference in New Issue
Block a user