diff options
author | Marc Mutz <marc.mutz@qt.io> | 2024-03-25 07:46:52 +0100 |
---|---|---|
committer | Marc Mutz <marc.mutz@qt.io> | 2024-03-25 16:22:05 +0100 |
commit | 73d00d05477fb6da25164b37135bb19a300bea8d (patch) | |
tree | ecde5c12f24c2d0b13768121a18f0f59d512c3fd /src/tools | |
parent | 990746096e21955fbb2b06e2848b3795eab6b328 (diff) |
syncqt: remove dead code
Either the code in the try {} block
- doesn't throw, then the return within the try {} block exits the
function, or it
- throws fileystem_error, then we rethrow in the catch, or it
- throws any other exception, then we leave the function by exception
In no case can control reach the trailing 'return {}'.
Found by Coverity.
Amends 7aecb189d544613ad13c7e7d1207bd7767225a71.
Pick-to: 6.7 6.6 6.5
Coverity-Id: 444466
Change-Id: I1c1bf752453076724c2fa9367ea5309e741d84ac
Reviewed-by: Alexey Edelev <alexey.edelev@qt.io>
Diffstat (limited to 'src/tools')
-rw-r--r-- | src/tools/syncqt/main.cpp | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/src/tools/syncqt/main.cpp b/src/tools/syncqt/main.cpp index db59a3c7ad..5df7b03fd5 100644 --- a/src/tools/syncqt/main.cpp +++ b/src/tools/syncqt/main.cpp @@ -151,7 +151,6 @@ std::filesystem::path normilizedPath(const std::string &path) printFilesystemError(fserr, "Unable to normalize path"); throw; } - return {}; } bool createDirectories(const std::string &path, std::string_view errorMsg, bool *exists = nullptr) |