[Libreoffice-commits] core.git: Branch 'distro/collabora/co-23.05' - filter/source

[Caolán McNamara (via logerrit)]

 filter/source/svg/svgwriter.cxx |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 948d12ac4e829fb6f815bc5849f0a8c018fe6407
Author:     Caolán McNamara <caolan.mcnam...@collabora.com>
AuthorDate: Wed Aug 16 15:04:58 2023 +0100
Commit:     Noel Grandin <noel.gran...@collabora.co.uk>
CommitDate: Thu Aug 17 08:19:54 2023 +0200

    cool#6616 compare graphics for equality with operator==...
    
    instead of checking equal CheckSums.
    
    For better or worse while Bitmap::operator== also considers equal
    CheckSums as equal Bitmaps operator== can quickly return true if
    both Bitmaps share the same underlying SalBitmap which is nearly
    always true in the example here
    
    Change-Id: I6b5cd484aeef7505721a493a3bff23ee345147dc
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/155706
    Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoff...@gmail.com>
    Reviewed-by: Noel Grandin <noel.gran...@collabora.co.uk>

diff --git a/filter/source/svg/svgwriter.cxx b/filter/source/svg/svgwriter.cxx
index 4df1352ddc10..bce563890361 100644
--- a/filter/source/svg/svgwriter.cxx
+++ b/filter/source/svg/svgwriter.cxx
@@ -2914,7 +2914,7 @@ void SVGActionWriter::ImplWriteBmp( const BitmapEx& 
rBmpEx,
         if (aGraphic.GetType() == GraphicType::Bitmap)
         {
             const BitmapEx& rGraphicBitmap = aGraphic.GetBitmapExRef();
-            if (rGraphicBitmap.GetChecksum() == rBmpEx.GetChecksum())
+            if (rGraphicBitmap == rBmpEx)
             {
                 bool bPNG = false;
                 GfxLink aGfxLink = aGraphic.GetGfxLink();