diff options
author | hjk <hjk@qt.io> | 2021-05-28 13:19:38 +0200 |
---|---|---|
committer | hjk <hjk@qt.io> | 2021-06-02 14:16:36 +0000 |
commit | 0ba43384671c00a4bad87b74012a184d91beb7c2 (patch) | |
tree | be3b6bc130b6119f628567d0714597690abe9d8f /src/plugins/git/gerrit/gerritserver.cpp | |
parent | 08040e4e94acba0c781b058ac5408955a4bc95fb (diff) | |
download | qt-creator-0ba43384671c00a4bad87b74012a184d91beb7c2.tar.gz |
Utils: Rename QtcProcess::Result::Finished to FinishedWithSuccess
To make clear that this is not just any finish.
Also change FinishedError to FinishedWithError, to create
symmetry.
Also adapt enum member description to reality.
Change-Id: I13e05391eb86fdb24e2ae660f14dfddb282e1104
Reviewed-by: Christian Kandeler <christian.kandeler@qt.io>
Diffstat (limited to 'src/plugins/git/gerrit/gerritserver.cpp')
-rw-r--r-- | src/plugins/git/gerrit/gerritserver.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/plugins/git/gerrit/gerritserver.cpp b/src/plugins/git/gerrit/gerritserver.cpp index 0944b7ee82..2429e16d97 100644 --- a/src/plugins/git/gerrit/gerritserver.cpp +++ b/src/plugins/git/gerrit/gerritserver.cpp @@ -247,7 +247,7 @@ int GerritServer::testConnection() SynchronousProcess proc; client->vcsFullySynchronousExec(proc, QString(), {curlBinary, arguments}, Core::ShellCommand::NoOutput); - if (proc.result() == QtcProcess::Finished) { + if (proc.result() == QtcProcess::FinishedWithSuccess) { QString output = proc.stdOut(); // Gerrit returns an empty response for /p/qt-creator/a/accounts/self // so consider this as 404. @@ -358,7 +358,7 @@ void GerritServer::resolveVersion(const GerritParameters &p, bool forceReload) Core::ShellCommand::NoOutput); // REST endpoint for version is only available from 2.8 and up. Do not consider invalid // if it fails. - if (proc.result() == QtcProcess::Finished) { + if (proc.result() == QtcProcess::FinishedWithSuccess) { QString output = proc.stdOut(); if (output.isEmpty()) return; |