Utils: Replace Result class by type alias to std::expected<T, QString>

... to be able to conveniently return also non-void cases without being
exposed to the syntax of expected.

The price for the more general approach is some uglification of the
void case: The previous 'Result' is now equivalent to 'Result<>',
which needs to be spelled out in function signatures, and some
changes to the special cases.

Change-Id: Ic5026e237ef2077a0765cdb8287122cae99d699f
Reviewed-by: Marcus Tillmanns <[email protected]>
Reviewed-by: Eike Ziller <[email protected]>

1 files changed, 3 insertions, 3 deletions

diff --git a/src/plugins/remotelinux/remotelinuxsignaloperation.cpp b/src/plugins/remotelinux/remotelinuxsignaloperation.cpp
index b48ea1f7504..0ba08f2d04e 100644
--- a/src/plugins/remotelinux/remotelinuxsignaloperation.cpp
+++ b/src/plugins/remotelinux/remotelinuxsignaloperation.cpp
@@ -77,11 +77,11 @@ void RemoteLinuxSignalOperation::interruptProcess(qint64 pid)
 
 void RemoteLinuxSignalOperation::runnerDone()
 {
-    Result result = Result::Ok;
+    Result<> result = ResultOk;
     if (m_process->exitStatus() != QProcess::NormalExit) {
-        result = Result::Error(m_process->errorString());
+        result = ResultError(m_process->errorString());
     } else if (m_process->exitCode() != 0) {
-        result = Result::Error(Tr::tr("Exit code is %1. stderr:").arg(m_process->exitCode())
+        result = ResultError(Tr::tr("Exit code is %1. stderr:").arg(m_process->exitCode())
                          + ' ' + QString::fromLatin1(m_process->rawStdErr()));
     }
     m_process.release()->deleteLater();