diff options
| author | Mike Kaganski <mike.kaganski@collabora.com> | 2018-12-02 20:46:02 +0300 |
|---|---|---|
| committer | Mike Kaganski <mike.kaganski@collabora.com> | 2018-12-03 01:20:21 +0100 |
| commit | eddee22b0eed069c3ee2929c9b36a9b4fb05f130 (patch) | |
| tree | 6ace29c24dfc58407049cef21d6af314f7624e14 /framework | |
| parent | 6ed52ecfed5781659cd27ff9aea5d0a04be77cd1 (diff) | |
tdf#120703 PVS: V560 A part of conditional expression is always true/false
Change-Id: Id7af9a6eaefc8b49a790eb299620c4fa97067a11
Reviewed-on: https://gerrit.libreoffice.org/64429
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
Diffstat (limited to 'framework')
| -rw-r--r-- | framework/source/services/substitutepathvars.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/framework/source/services/substitutepathvars.cxx b/framework/source/services/substitutepathvars.cxx index afa83643b87a..13dc84760517 100644 --- a/framework/source/services/substitutepathvars.cxx +++ b/framework/source/services/substitutepathvars.cxx @@ -345,7 +345,7 @@ OUString SubstitutePathVariables::impl_substituteVariable( const OUString& rText // Is there something to replace ? bool bWorkRetrieved = false; bool bWorkDirURLRetrieved = false; - while ( !bSubstitutionCompleted && nDepth < nMaxRecursiveDepth ) + while (nDepth < nMaxRecursiveDepth) { while ( ( nPosition != -1 ) && ( nLength > 3 ) ) // "$(" ")" { |