diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-09-27 11:36:18 +0200 |
|---|---|---|
| committer | Ashod Nakashian <ashod.nakashian@collabora.co.uk> | 2017-12-05 08:19:26 -0500 |
| commit | c30f0d7eb04e6ddc9c1522a745e3bd3bdd622643 (patch) | |
| tree | 2ac28cb5bd78b02cbf342ea5fde5cdccebfc1409 /cui | |
| parent | d87778376321abac9de5ad4bc79bf2b35c679738 (diff) | |
simplify Bitmap operator==/IsEqual (stage1)
It looks like operator== is actually only useful internally. So inline
that, and rename the other call sites to use IsEqual().
As a second stage, I will rename IsEqual to operator==, the intention
being to make it obvious how the call sites are modified.
Change-Id: I37f2920a8cafaffb25e8c5c16e6559546206684b
Reviewed-on: https://gerrit.libreoffice.org/42846
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
(cherry picked from commit 404630c6052919c1a0b5dab405084c2a39fc2f5c)
Reviewed-on: https://gerrit.libreoffice.org/45455
Reviewed-by: Ashod Nakashian <ashnakash@gmail.com>
Tested-by: Ashod Nakashian <ashnakash@gmail.com>
(cherry picked from commit 00f776ba53975fea14faa78ffa7b06e7bf5fbf6b)
Diffstat (limited to 'cui')
| -rw-r--r-- | cui/source/tabpages/backgrnd.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/cui/source/tabpages/backgrnd.cxx b/cui/source/tabpages/backgrnd.cxx index 43d8fdd2eddb..2b908310596d 100644 --- a/cui/source/tabpages/backgrnd.cxx +++ b/cui/source/tabpages/backgrnd.cxx @@ -716,7 +716,7 @@ bool SvxBackgroundTabPage::FillItemSet( SfxItemSet* rCoreSet ) { const Graphic* pGraphic = rOldItem.GetGraphic(); if (pGraphic) - bModifyBrush = pGraphic->GetBitmap() != aBgdGraphic.GetBitmap(); + bModifyBrush = !pGraphic->GetBitmap().IsEqual(aBgdGraphic.GetBitmap()); } if (bModifyBrush) { |