diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-10-19 08:12:17 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-10-19 10:02:39 +0200 |
| commit | 2c1b7e8d6a7fa22cb91919238418816671c3a497 (patch) | |
| tree | 6f356017d24dffcd25261295ab25a21b738dc835 /sw/source/ui | |
| parent | ad18bb24d51e4f735085d50c496d28bd637dbb0b (diff) | |
clang-tidy readability-container-size-empty
Change-Id: I1df70b7dff5ebb6048f7fc618789faa15ca5d422
Reviewed-on: https://gerrit.libreoffice.org/61967
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'sw/source/ui')
| -rw-r--r-- | sw/source/ui/dbui/createaddresslistdialog.cxx | 6 | ||||
| -rw-r--r-- | sw/source/ui/dbui/mmoutputtypepage.cxx | 2 |
2 files changed, 4 insertions, 4 deletions
diff --git a/sw/source/ui/dbui/createaddresslistdialog.cxx b/sw/source/ui/dbui/createaddresslistdialog.cxx index 736c9adc0321..ad8452df3e5d 100644 --- a/sw/source/ui/dbui/createaddresslistdialog.cxx +++ b/sw/source/ui/dbui/createaddresslistdialog.cxx @@ -135,7 +135,7 @@ void SwAddressControl_Impl::SetData(SwCSVData& rDBData) { m_pData = &rDBData; //when the address data is updated then remove the controls an build again - if(m_aFixedTexts.size()) + if(!m_aFixedTexts.empty()) { for(auto aTextIter = m_aFixedTexts.begin(); aTextIter != m_aFixedTexts.end(); ++aTextIter) aTextIter->disposeAndClear(); @@ -233,7 +233,7 @@ void SwAddressControl_Impl::SetData(SwCSVData& rDBData) } // Even if no items in m_aEdits, the scrollbar will still exist; // we might as well disable it. - if (m_aEdits.size() < 1) { + if (m_aEdits.empty()) { m_pScrollBar->DoScroll(0); m_pScrollBar->SetThumbPos(0); m_pScrollBar->Enable(false); @@ -377,7 +377,7 @@ void SwAddressControl_Impl::Resize() m_pWindow->SetSizePixel(Size(aSize.Width() - nScrollBarWidth, m_pWindow->GetOutputSizePixel().Height())); m_pScrollBar->SetPosPixel(Point(aSize.Width() - nScrollBarWidth, 0)); - if(m_aEdits.size()) + if(!m_aEdits.empty()) { long nNewEditSize = aSize.Width() - (*m_aEdits.begin())->GetPosPixel().X() - nScrollBarWidth - 6; diff --git a/sw/source/ui/dbui/mmoutputtypepage.cxx b/sw/source/ui/dbui/mmoutputtypepage.cxx index ae0171148c8a..86f0a1a1dac1 100644 --- a/sw/source/ui/dbui/mmoutputtypepage.cxx +++ b/sw/source/ui/dbui/mmoutputtypepage.cxx @@ -552,7 +552,7 @@ void SwSendMailDialog::UpdateTransferStatus() sStatus = m_sErrorStatus.replaceFirst("%1", OUString::number(m_nErrorCount) ); m_pErrorStatus->SetText(sStatus); - if (m_pImpl->aDescriptors.size()) + if (!m_pImpl->aDescriptors.empty()) { assert(m_nExpectedCount && "div-by-zero"); m_pProgressBar->SetValue(static_cast<sal_uInt16>(m_nSendCount * 100 / m_nExpectedCount)); |