diff options
author | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-02-16 11:56:18 +0200 |
---|---|---|
committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-02-16 15:13:53 +0100 |
commit | 1d180f48bc61444c610c35083e35745d38c6cce2 (patch) | |
tree | e580741aa902908a9fcf0c4b224e93ff42da1405 /fpicker | |
parent | svl: fix loplugin:staticanonymous warning (diff) | |
download | core-1d180f48bc61444c610c35083e35745d38c6cce2.tar.gz core-1d180f48bc61444c610c35083e35745d38c6cce2.zip |
Revert "loplugin:changetoolsgen in filter..framework" and reapply plugin
because I
(a) forgot to insert parentheses which changes the meaning of some expressions and
(b) I now use the AdjustFoo calls when changing unary operations, which reads much better
This reverts commit bf8a6cacd06d7c1961b5db8cc79ee4bbdb342e43.
Change-Id: I4b5a9bf0c38ee1b57af91e6b7f184f1e8807f6f7
Reviewed-on: https://gerrit.libreoffice.org/49843
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'fpicker')
-rw-r--r-- | fpicker/source/office/PlacesListBox.cxx | 2 | ||||
-rw-r--r-- | fpicker/source/office/RemoteFilesDialog.cxx | 2 | ||||
-rw-r--r-- | fpicker/source/office/iodlg.cxx | 2 |
3 files changed, 3 insertions, 3 deletions
diff --git a/fpicker/source/office/PlacesListBox.cxx b/fpicker/source/office/PlacesListBox.cxx index b4ea90e3bdd4..8a7156c1019b 100644 --- a/fpicker/source/office/PlacesListBox.cxx +++ b/fpicker/source/office/PlacesListBox.cxx @@ -160,7 +160,7 @@ void PlacesListBox::SetSizePixel( const Size& rNewSize ) { Control::SetSizePixel( rNewSize ); Size aListSize( rNewSize ); - aListSize.setHeight( aListSize.Height() - 26 + 18 ); + aListSize.AdjustHeight( -(26 + 18) ); mpImpl->SetSizePixel( aListSize ); sal_Int32 nBtnY = rNewSize.Height() - 26; diff --git a/fpicker/source/office/RemoteFilesDialog.cxx b/fpicker/source/office/RemoteFilesDialog.cxx index 971f409b563e..d70723eb7634 100644 --- a/fpicker/source/office/RemoteFilesDialog.cxx +++ b/fpicker/source/office/RemoteFilesDialog.cxx @@ -1033,7 +1033,7 @@ IMPL_LINK_NOARG( RemoteFilesDialog, SplitHdl, Splitter*, void ) sal_Int32 nNewX = nSplitPos + m_pSplitter->GetSizePixel().Width(); fileViewPos.setX( nNewX ); Size fileViewSize = m_pFileView->GetSizePixel(); - fileViewSize.setWidth( fileViewSize.Width() - ( nNewX - nOldX ) ); + fileViewSize.AdjustWidth( -( nNewX - nOldX ) ); m_pFileView->SetPosSizePixel( fileViewPos, fileViewSize ); m_pSplitter->SetPosPixel( Point( placeSize.Width(), m_pSplitter->GetPosPixel().Y() ) ); diff --git a/fpicker/source/office/iodlg.cxx b/fpicker/source/office/iodlg.cxx index 647879d9c5eb..f8b5e1dd5d7f 100644 --- a/fpicker/source/office/iodlg.cxx +++ b/fpicker/source/office/iodlg.cxx @@ -2755,7 +2755,7 @@ IMPL_LINK_NOARG( SvtFileDialog, Split_Hdl, Splitter*, void ) sal_Int32 nNewX = nSplitPos + _pSplitter->GetSizePixel().Width(); fileViewPos.setX( nNewX ); Size fileViewSize = _pFileView->GetSizePixel(); - fileViewSize.setWidth( fileViewSize.Width() - ( nNewX - nOldX ) ); + fileViewSize.AdjustWidth( -( nNewX - nOldX ) ); _pFileView->SetPosSizePixel( fileViewPos, fileViewSize ); _pSplitter->SetPosPixel( Point( placeSize.Width(), _pSplitter->GetPosPixel().Y() ) ); |