simplify Bitmap operator==/IsEqual (stage2)

Now that we've sorted out which bits are rightfully internal (see commit
7a4064cf87d23bffac987b3e39bf87d9b4d158fc), and which bits need a
comparison operator, rename IsEqual to operator==.

Change-Id: Iec229a7571d76ee18fbc0a4fe672cc2e50a44948
Reviewed-on: https://gerrit.libreoffice.org/42851Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>

cui/source/tabpages/backgrnd.cxx

@@ -683,7 +683,7 @@ bool SvxBackgroundTabPage::FillItemSet( SfxItemSet* rCoreSet )
                 {
                     const Graphic* pGraphic = rOldItem.GetGraphic();
                     if (pGraphic)
-                        bModifyBrush = !pGraphic->GetBitmap().IsEqual(aBgdGraphic.GetBitmap());
+                        bModifyBrush = pGraphic->GetBitmap() != aBgdGraphic.GetBitmap();
                 }
                 if (bModifyBrush)
                 {

include/vcl/alpha.hxx

@@ -44,9 +44,8 @@ public:
     AlphaMask&  operator=( const AlphaMask& rAlphaMask ) { return static_cast<AlphaMask&>( Bitmap::operator=( rAlphaMask ) ); }
     AlphaMask&  operator=( AlphaMask&& rAlphaMask ) { return static_cast<AlphaMask&>( Bitmap::operator=( std::move(rAlphaMask) ) ); }
     bool        operator!() const { return Bitmap::operator!(); }
-    bool        operator==( const AlphaMask& rAlphaMask ) const = delete;
-    bool        operator!=( const AlphaMask& rAlphaMask ) const = delete;
-    bool        IsEqual( const AlphaMask& rAlphaMask ) const { return Bitmap::IsEqual(rAlphaMask); }
+    bool        operator==( const AlphaMask& rAlphaMask ) const { return Bitmap::operator==(rAlphaMask); }
+    bool        operator!=( const AlphaMask& rAlphaMask ) const { return Bitmap::operator!=(rAlphaMask); }
 
     void        SetPrefMapMode( const MapMode& rMapMode ) { Bitmap::SetPrefMapMode( rMapMode ); }
 

include/vcl/bitmap.hxx

@@ -224,10 +224,8 @@ public:
     Bitmap&                 operator=( const Bitmap& rBitmap );
     Bitmap&                 operator=( Bitmap&& rBitmap );
     inline bool             operator!() const;
-    bool                    operator==( const Bitmap& rBitmap ) const = delete;
-    bool                    operator!=( const Bitmap& rBitmap ) const = delete;
-
-    bool                    IsEqual( const Bitmap& rBmp ) const;
+    bool                    operator==( const Bitmap& rBitmap ) const;
+    bool                    operator!=( const Bitmap& rBitmap ) const { return !operator==(rBitmap); }
 
     inline bool             IsEmpty() const;
     void                    SetEmpty();

svx/source/sidebar/nbdtmg.cxx

@@ -670,8 +670,8 @@ sal_uInt16 OutlineTypeMgr::GetNBOIndexForNumRule(SvxNumRule& aNum,sal_uInt16 /*m
                             const Graphic* pGrf2 = pBrsh2->GetGraphic();
                             if (pGrf1==pGrf2) bIsMatch = true;
                             if (pGrf1 && pGrf2) {
-                                if ( pGrf1->GetBitmap().IsEqual(pGrf2->GetBitmap()) &&
-                                     _pSet->aSize==aFmt.GetGraphicSize())
+                                if ( pGrf1->GetBitmap() == pGrf2->GetBitmap() &&
+                                     _pSet->aSize == aFmt.GetGraphicSize())
                                     bIsMatch = true;
                             }
                         }

vcl/source/gdi/bitmap.cxx

@@ -231,7 +231,7 @@ Bitmap& Bitmap::operator=( Bitmap&& rBitmap )
     return *this;
 }
 
-bool Bitmap::IsEqual( const Bitmap& rBmp ) const
+bool Bitmap::operator==( const Bitmap& rBmp ) const
 {
     return rBmp.mxImpBmp == mxImpBmp || // Includes both are nullptr
            (rBmp.mxImpBmp && mxImpBmp && mxImpBmp->ImplIsEqual(*rBmp.mxImpBmp));

vcl/source/gdi/bitmapex.cxx

@@ -193,7 +193,7 @@ bool BitmapEx::operator==( const BitmapEx& rBitmapEx ) const
     if( eTransparent != rBitmapEx.eTransparent )
         return false;
 
-    if( !aBitmap.IsEqual(rBitmapEx.aBitmap) )
+    if( aBitmap != rBitmapEx.aBitmap )
         return false;
 
     if( aBitmapSize != rBitmapEx.aBitmapSize )
@@ -205,7 +205,7 @@ bool BitmapEx::operator==( const BitmapEx& rBitmapEx ) const
     if( eTransparent == TransparentType::Color )
         return aTransparentColor == rBitmapEx.aTransparentColor;
 
-    return aMask.IsEqual(rBitmapEx.aMask) && bAlpha == rBitmapEx.bAlpha;
+    return aMask == rBitmapEx.aMask && bAlpha == rBitmapEx.bAlpha;
 }
 
 bool BitmapEx::IsEmpty() const