diff options
| author | Mike Kaganski <mike.kaganski@collabora.com> | 2018-11-04 17:06:47 +0300 |
|---|---|---|
| committer | Mike Kaganski <mike.kaganski@collabora.com> | 2018-11-04 16:57:59 +0100 |
| commit | 628e96ce1fce194b30a5c2912feb8b7ec0328db0 (patch) | |
| tree | 7a901d6da7d0dea1da3ac55ebe96fc33fc904150 /stoc/source | |
| parent | 9d5365f7cacecf92f1f056dce7bb6bbaf9f0cc40 (diff) | |
tdf#120703 PVS: V547 Expression is always true/false
Change-Id: I3432afca1ee9bf9e8adce1d55d58d57bf1a09cb4
Reviewed-on: https://gerrit.libreoffice.org/62847
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
Diffstat (limited to 'stoc/source')
| -rw-r--r-- | stoc/source/corereflection/lrucache.hxx | 15 |
1 files changed, 6 insertions, 9 deletions
diff --git a/stoc/source/corereflection/lrucache.hxx b/stoc/source/corereflection/lrucache.hxx index e9a89bebd66f..b52a05fe1213 100644 --- a/stoc/source/corereflection/lrucache.hxx +++ b/stoc/source/corereflection/lrucache.hxx @@ -90,16 +90,13 @@ inline LRU_Cache< t_Key, t_Val, t_KeyHash >::LRU_Cache() , _pHead( nullptr ) , _pTail( nullptr ) { - if (_nCachedElements > 0) + _pBlock.reset(new CacheEntry[_nCachedElements]); + _pHead = _pBlock.get(); + _pTail = _pBlock.get() + _nCachedElements - 1; + for (sal_Int32 nPos = _nCachedElements; nPos--;) { - _pBlock.reset(new CacheEntry[_nCachedElements]); - _pHead = _pBlock.get(); - _pTail = _pBlock.get() + _nCachedElements -1; - for ( sal_Int32 nPos = _nCachedElements; nPos--; ) - { - _pBlock[nPos].pPred = _pBlock.get() + nPos -1; - _pBlock[nPos].pSucc = _pBlock.get() + nPos +1; - } + _pBlock[nPos].pPred = _pBlock.get() + nPos - 1; + _pBlock[nPos].pSucc = _pBlock.get() + nPos + 1; } } |