diff options
author | Denis Shienkov <denis.shienkov@gmail.com> | 2019-02-26 14:38:54 +0300 |
---|---|---|
committer | Denis Shienkov <denis.shienkov@gmail.com> | 2019-02-26 15:52:26 +0000 |
commit | 4fd17d627106fde01284075038e15cc0680611bc (patch) | |
tree | 25c5b7e8ec774d362ad97e86d0ecd1d8527fbac5 /src/lib/corelib/tools/version.cpp | |
parent | 0b8f0b771080e51a59664782ced6b3a1cc5111ca (diff) |
Return initializer list where it is possible
This fixes this clang-tidy warning:
warning: avoid repeating the return type from the declaration; use a braced initializer list instead [modernize-return-braced-init-list]
Change-Id: I421e1e47462fe0e97788672684d47943af7df850
Reviewed-by: Christian Kandeler <christian.kandeler@qt.io>
Diffstat (limited to 'src/lib/corelib/tools/version.cpp')
-rw-r--r-- | src/lib/corelib/tools/version.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/lib/corelib/tools/version.cpp b/src/lib/corelib/tools/version.cpp index 264c44c29..ccc9dd799 100644 --- a/src/lib/corelib/tools/version.cpp +++ b/src/lib/corelib/tools/version.cpp @@ -97,12 +97,12 @@ Version Version::fromString(const QString &versionString, bool buildNumberAllowe pattern += QStringLiteral("(?:-(\\d+))?"); // And possibly a dash followed by the build number. QRegExp rex(pattern); if (!rex.exactMatch(versionString)) - return Version(); + return Version{}; const int majorNr = rex.cap(1).toInt(); const int minorNr = rex.captureCount() >= 2 ? rex.cap(2).toInt() : 0; const int patchNr = rex.captureCount() >= 3 ? rex.cap(3).toInt() : 0; const int buildNr = rex.captureCount() >= 4 ? rex.cap(4).toInt() : 0; - return Version(majorNr, minorNr, patchNr, buildNr); + return Version{majorNr, minorNr, patchNr, buildNr}; } QString Version::toString() const |