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, 1 insertions, 1 deletions

diff --git a/src/plugins/android/androidbuildapkstep.cpp b/src/plugins/android/androidbuildapkstep.cpp
index 0a7a9d03c2e..5d0bf2b5d11 100644
--- a/src/plugins/android/androidbuildapkstep.cpp
+++ b/src/plugins/android/androidbuildapkstep.cpp
@@ -892,7 +892,7 @@ static bool copyFileIfNewer(const FilePath &sourceFilePath, const FilePath &dest
 
     if (!destinationFilePath.parentDir().ensureWritableDir())
         return false;
-    Result result = sourceFilePath.copyFile(destinationFilePath);
+    Result<> result = sourceFilePath.copyFile(destinationFilePath);
     QTC_ASSERT_EXPECTED(result, return false);
     return true;
 }