diff options
author | Noel Grandin <noel@peralex.com> | 2016-02-02 16:15:51 +0200 |
---|---|---|
committer | Noel Grandin <noelgrandin@gmail.com> | 2016-02-08 06:08:14 +0000 |
commit | b14224fe97b8a44232c9c1401d3a49771f46582e (patch) | |
tree | 8f9cf31cf4b51a0edbb43022499a6acd17d0945d /vcl | |
parent | c474e610e453d0f38f7cc6cb9559ad7e7b5d69ca (diff) |
loplugin:unusedmethods
using an idea from dtardon:
<dtardon> noelgrandin, hi. could you try to run the unusedmethods clang
plugin with "make build-nocheck"? that would catch functions that are
only used in tests. e.g., i just removed the whole o3tl::range class,
which has not been used in many years, but htere was a test for it...
<noelgrandin> dtardon, interesting idea! Sure, I can do that.
Change-Id: I5653953a426a2186a1e43017212d87ffce520387
Reviewed-on: https://gerrit.libreoffice.org/22041
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
Tested-by: Noel Grandin <noelgrandin@gmail.com>
Diffstat (limited to 'vcl')
-rw-r--r-- | vcl/source/app/settings.cxx | 12 | ||||
-rw-r--r-- | vcl/source/gdi/dibtools.cxx | 5 | ||||
-rw-r--r-- | vcl/source/gdi/print.cxx | 11 | ||||
-rw-r--r-- | vcl/source/window/menu.cxx | 15 | ||||
-rw-r--r-- | vcl/source/window/window.cxx | 46 |
5 files changed, 0 insertions, 89 deletions
diff --git a/vcl/source/app/settings.cxx b/vcl/source/app/settings.cxx index 296684b3dfd9..d952dd7f801e 100644 --- a/vcl/source/app/settings.cxx +++ b/vcl/source/app/settings.cxx @@ -2678,12 +2678,6 @@ HelpSettings::GetBalloonDelay() const return mxData->mnBalloonDelay; } -bool -HelpSettings::operator !=( const HelpSettings& rSet ) const -{ - return !(*this == rSet); -} - ImplAllSettingsData::ImplAllSettingsData() : maLocale( LANGUAGE_SYSTEM ), @@ -3133,12 +3127,6 @@ AllSettings::GetWindowUpdate() const return mxData->mnWindowUpdate; } -bool -AllSettings::operator !=( const AllSettings& rSet ) const -{ - return !(*this == rSet); -} - SvtSysLocale& AllSettings::GetSysLocale() { diff --git a/vcl/source/gdi/dibtools.cxx b/vcl/source/gdi/dibtools.cxx index a1b05157fadb..84904bdb0f38 100644 --- a/vcl/source/gdi/dibtools.cxx +++ b/vcl/source/gdi/dibtools.cxx @@ -1765,11 +1765,6 @@ bool WriteDIBBitmapEx( return false; } -sal_uInt32 getDIBInfoHeaderSize() -{ - return DIBINFOHEADERSIZE; -} - sal_uInt32 getDIBV5HeaderSize() { return DIBV5HEADERSIZE; diff --git a/vcl/source/gdi/print.cxx b/vcl/source/gdi/print.cxx index 0dca9afda244..2d3cfe06271a 100644 --- a/vcl/source/gdi/print.cxx +++ b/vcl/source/gdi/print.cxx @@ -362,17 +362,6 @@ QueueInfo::~QueueInfo() { } -bool QueueInfo::operator==( const QueueInfo& rInfo ) const -{ - return - maPrinterName == rInfo.maPrinterName && - maDriver == rInfo.maDriver && - maLocation == rInfo.maLocation && - maComment == rInfo.maComment && - mnStatus == rInfo.mnStatus && - mnJobs == rInfo.mnJobs; -} - SalPrinterQueueInfo::SalPrinterQueueInfo() { mnStatus = PrintQueueFlags::NONE; diff --git a/vcl/source/window/menu.cxx b/vcl/source/window/menu.cxx index d14d1a3584ef..2e7499ed2188 100644 --- a/vcl/source/window/menu.cxx +++ b/vcl/source/window/menu.cxx @@ -2511,21 +2511,6 @@ void MenuBar::ClosePopup(Menu *pMenu) pMenuWin->PopupClosed(pMenu); } -void MenuBar::DeactivateMenuBar(VclPtr<vcl::Window>& xFocusId) -{ - MenuBarWindow* pMenuWin = getMenuBarWindow(); - if (!pMenuWin) - { - xFocusId = nullptr; - return; - } - xFocusId = pMenuWin->GetFocusId(); - if (xFocusId == nullptr) - return; - pMenuWin->SetFocusId(nullptr); - ImplGetSVData()->maWinData.mbNoDeactivate = false; -} - void MenuBar::MenuBarKeyInput(const KeyEvent& rEvent) { pWindow->KeyInput(rEvent); diff --git a/vcl/source/window/window.cxx b/vcl/source/window/window.cxx index 1d04005eb0ef..599514d66354 100644 --- a/vcl/source/window/window.cxx +++ b/vcl/source/window/window.cxx @@ -1212,52 +1212,6 @@ void Window::ImplInitWindowData( WindowType nType ) mbEnableRTL = AllSettings::GetLayoutRTL(); // true: this outdev will be mirrored if RTL window layout (UI mirroring) is globally active } -void Window::getFrameDev( const Point& rPt, const Point& rDevPt, const Size& rDevSize, OutputDevice& rDev ) -{ - bool bOldMap = mbMap; - mbMap = false; - rDev.DrawOutDev( rDevPt, rDevSize, rPt, rDevSize, *this ); - mbMap = bOldMap; -} - -void Window::drawFrameDev( const Point& rPt, const Point& rDevPt, const Size& rDevSize, - const OutputDevice& rOutDev, const vcl::Region& rRegion ) -{ - - GDIMetaFile* pOldMetaFile = mpMetaFile; - bool bOldMap = mbMap; - RasterOp eOldROP = GetRasterOp(); - mpMetaFile = nullptr; - mbMap = false; - SetRasterOp( ROP_OVERPAINT ); - - if ( !IsDeviceOutputNecessary() ) - return; - - if ( !mpGraphics ) - { - if ( !AcquireGraphics() ) - return; - } - - if ( rRegion.IsNull() ) - mpGraphics->ResetClipRegion(); - else - SelectClipRegion( rRegion ); - - SalTwoRect aPosAry(rDevPt.X(), rDevPt.Y(), rDevSize.Width(), rDevSize.Height(), - rPt.X(), rPt.Y(), rDevSize.Width(), rDevSize.Height()); - drawOutDevDirect( &rOutDev, aPosAry ); - - // Ensure that ClipRegion is recalculated and set - mbInitClipRegion = true; - - SetRasterOp( eOldROP ); - mbMap = bOldMap; - mpMetaFile = pOldMetaFile; -} - - ImplWinData* Window::ImplGetWinData() const { if ( !mpWindowImpl->mpWinData ) |