diff options
| author | Caolán McNamara <caolanm@redhat.com> | 2015-09-11 09:19:46 +0100 |
|---|---|---|
| committer | Caolán McNamara <caolanm@redhat.com> | 2015-09-11 09:31:45 +0100 |
| commit | 0f36d4a923f37b1fd85ab024d09e84e53bf481ba (patch) | |
| tree | 7bb0ecda97fdf4835d12fb81559fcdf66f866f10 /canvas | |
| parent | 3c319565a5d92ba7082c28fe01a42115fcb7a9a8 (diff) | |
coverity#1323755 Using invalid iterator
try and help coverity to know that aEnd and maFragments.end() are the same
place
Change-Id: I9ce4187d5ed15d0ae26b69bb360dbd47c2a6a185
Diffstat (limited to 'canvas')
| -rw-r--r-- | canvas/source/tools/pagemanager.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/canvas/source/tools/pagemanager.cxx b/canvas/source/tools/pagemanager.cxx index 46c57167a4a5..3d08eff21593 100644 --- a/canvas/source/tools/pagemanager.cxx +++ b/canvas/source/tools/pagemanager.cxx @@ -91,7 +91,7 @@ namespace canvas // TODO(F1): this is a heuristic, could // be designed as a policy. auto aEnd( maFragments.cend() ); - auto aCurrMax( maFragments.end() ); + auto aCurrMax( aEnd ); sal_uInt32 nCurrMaxArea = 0; for( auto aCurr = maFragments.begin(); aCurr != aEnd; ++aCurr ) { |