diff options
author | Mike Kaganski <mike.kaganski@collabora.com> | 2018-11-26 00:07:33 +0300 |
---|---|---|
committer | Mike Kaganski <mike.kaganski@collabora.com> | 2018-11-26 06:30:58 +0100 |
commit | 761546e088dcccf3aa66c3f5c4853f110b93770f (patch) | |
tree | 94dc648b78e78d685aab8b8b8e41b370a5b9db54 /desktop/source | |
parent | partial revert tdf79435 doc: round-trip legacy input formfields (diff) | |
download | core-761546e088dcccf3aa66c3f5c4853f110b93770f.tar.gz core-761546e088dcccf3aa66c3f5c4853f110b93770f.zip |
tdf#120703 PVS: V560 A part of conditional expression is always true/false
Change-Id: I1ac4c52ea51503373644bc58dcd4395c69f1a675
Reviewed-on: https://gerrit.libreoffice.org/64007
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
Diffstat (limited to 'desktop/source')
-rw-r--r-- | desktop/source/migration/migration.cxx | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/desktop/source/migration/migration.cxx b/desktop/source/migration/migration.cxx index ce32432e2dcf..4bec38e38bda 100644 --- a/desktop/source/migration/migration.cxx +++ b/desktop/source/migration/migration.cxx @@ -339,7 +339,8 @@ static void insertSorted(migrations_available& rAvailableMigrations, supported_m { bool bInserted( false ); migrations_available::iterator pIter = rAvailableMigrations.begin(); - while ( !bInserted && pIter != rAvailableMigrations.end()) { + while (pIter != rAvailableMigrations.end()) + { if ( pIter->nPriority < aSupportedMigration.nPriority ) { rAvailableMigrations.insert(pIter, aSupportedMigration ); bInserted = true; |