diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-10-03 12:47:53 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-10-03 15:29:19 +0200 |
| commit | 6725311151403e5bf5bef44e08c35ac76527aba4 (patch) | |
| tree | 62dda4a35d0cc2b8f96be6063e9c66c28957a0f4 /io | |
| parent | 0de15abda691b78bef38862f078e60d97da549a5 (diff) | |
no need to call empty() before clear()
found with
git grep -A2 -nP '!.*\.empty()' | grep -B1 -w clear
Change-Id: I87013eab9c6988048b891d20577e1f7efbce1b6c
Reviewed-on: https://gerrit.libreoffice.org/61295
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'io')
| -rw-r--r-- | io/source/stm/omark.cxx | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/io/source/stm/omark.cxx b/io/source/stm/omark.cxx index 989c20894b0f..af6b7d390f3f 100644 --- a/io/source/stm/omark.cxx +++ b/io/source/stm/omark.cxx @@ -179,10 +179,7 @@ void OMarkableOutputStream::closeOutput() MutexGuard guard( m_mutex ); // all marks must be cleared and all - if( ! m_mapMarks.empty() ) - { - m_mapMarks.clear(); - } + m_mapMarks.clear(); m_nCurrentPos = m_pBuffer->getSize(); checkMarksAndFlush(); |