diff options
| author | Caolán McNamara <caolanm@redhat.com> | 2014-11-05 10:15:38 +0000 |
|---|---|---|
| committer | Caolán McNamara <caolanm@redhat.com> | 2014-11-05 14:58:00 +0000 |
| commit | 12bcfec04fcbe6425e327109ad47cd2b2b80d2bd (patch) | |
| tree | b57ac6784ce892defa6d06fd5974318c8ffb6972 /svl | |
| parent | ea272d9fe4bf2dc7419f76e2ddee707d768ccd50 (diff) | |
markup with event type not checker type
Change-Id: I14c0c5d90b67000cb4fe9e6be647854abfe784da
Diffstat (limited to 'svl')
| -rw-r--r-- | svl/source/items/whiter.cxx | 10 | ||||
| -rw-r--r-- | svl/source/misc/strmadpt.cxx | 1 |
2 files changed, 3 insertions, 8 deletions
diff --git a/svl/source/items/whiter.cxx b/svl/source/items/whiter.cxx index 9a4e748a62ad..d69960c6f628 100644 --- a/svl/source/items/whiter.cxx +++ b/svl/source/items/whiter.cxx @@ -26,18 +26,14 @@ SfxWhichIter::SfxWhichIter( const SfxItemSet& rSet, sal_uInt16 nFromWh, sal_uInt pStart(rSet.GetRanges()), nOfst(0), nFrom(nFromWh), nTo(nToWh) { - if ( nFrom > 0 ) - FirstWhich(); + if (nFrom > 0) + (void)FirstWhich(); } - - SfxWhichIter::~SfxWhichIter() { } - - sal_uInt16 SfxWhichIter::NextWhich() { while ( 0 != *pRanges ) @@ -56,8 +52,6 @@ sal_uInt16 SfxWhichIter::NextWhich() return 0; } - - sal_uInt16 SfxWhichIter::FirstWhich() { pRanges = pStart; diff --git a/svl/source/misc/strmadpt.cxx b/svl/source/misc/strmadpt.cxx index 7b8dc4e05168..207e52137668 100644 --- a/svl/source/misc/strmadpt.cxx +++ b/svl/source/misc/strmadpt.cxx @@ -945,6 +945,7 @@ bool SvDataPipe_Impl::removeMark(sal_uInt32 nPosition) if (t == m_aMarks.end()) return false; m_aMarks.erase(t); + // coverity[pass_freed_arg] - remove frees m_pFirstPage but then sets m_pFirstPage to something else while (remove(m_pFirstPage)) ; return true; } |