From 404630c6052919c1a0b5dab405084c2a39fc2f5c Mon Sep 17 00:00:00 2001 From: Noel Grandin Date: Wed, 27 Sep 2017 11:36:18 +0200 Subject: 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 Reviewed-by: Noel Grandin --- cui/source/tabpages/backgrnd.cxx | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'cui') diff --git a/cui/source/tabpages/backgrnd.cxx b/cui/source/tabpages/backgrnd.cxx index 6d483aecb0f1..e93ab92180f7 100644 --- a/cui/source/tabpages/backgrnd.cxx +++ b/cui/source/tabpages/backgrnd.cxx @@ -683,7 +683,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) { -- cgit