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, 2 insertions, 3 deletions

diff --git a/src/plugins/baremetal/idebugserverprovider.h b/src/plugins/baremetal/idebugserverprovider.h
index 4f21d70d275..375e2021a35 100644
--- a/src/plugins/baremetal/idebugserverprovider.h
+++ b/src/plugins/baremetal/idebugserverprovider.h
@@ -9,6 +9,7 @@
 
 #include <utils/filepath.h>
 #include <utils/store.h>
+#include <utils/result.h>
 
 #include <QSet>
 #include <QUrl>
@@ -28,8 +29,6 @@ class RunControl;
 class RunWorker;
 }
 
-namespace Utils { class Result; }
-
 namespace BareMetal::Internal {
 
 class BareMetalDevice;
@@ -69,7 +68,7 @@ public:
     virtual void toMap(Utils::Store &data) const;
     virtual void fromMap(const Utils::Store &data);
 
-    virtual Utils::Result setupDebuggerRunParameters(Debugger::DebuggerRunParameters &rp,
+    virtual Utils::Result<> setupDebuggerRunParameters(Debugger::DebuggerRunParameters &rp,
                                                      ProjectExplorer::RunControl *runControl) const = 0;
     virtual ProjectExplorer::RunWorker *targetRunner(
             ProjectExplorer::RunControl *runControl) const = 0;