vcl/source/bitmap/bitmap.cxx |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit ea3eefae42be0bdbf4764f307e51b842ca06fcbe
Author:     Noel Grandin <noelgran...@gmail.com>
AuthorDate: Sat Sep 6 09:08:19 2025 +0200
Commit:     Noel Grandin <noelgran...@gmail.com>
CommitDate: Sat Sep 6 16:30:59 2025 +0200

    improve assert in Bitmap::SplitIntoColorAndAlpha
    
    N32_BPP does not necessary mean that we have an alpha channel
    
    Change-Id: Ib68a3a03325af19f8befe46fa045bbfad22f2db6
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/190630
    Tested-by: Jenkins
    Reviewed-by: Noel Grandin <noel.gran...@collabora.co.uk>

diff --git a/vcl/source/bitmap/bitmap.cxx b/vcl/source/bitmap/bitmap.cxx
index 54f40d34d2a4..ec0863c9bb70 100644
--- a/vcl/source/bitmap/bitmap.cxx
+++ b/vcl/source/bitmap/bitmap.cxx
@@ -1821,7 +1821,7 @@ const basegfx::SystemDependentDataHolder* 
Bitmap::accessSystemDependentDataHolde
 
 std::pair<Bitmap, AlphaMask> Bitmap::SplitIntoColorAndAlpha() const
 {
-    assert(getPixelFormat() == vcl::PixelFormat::N32_BPP && "only valid to 
call this when this is a 32-bit combined color+alpha bitmap");
+    assert(HasAlpha() && "only valid to call this when this is a 32-bit 
combined color+alpha bitmap");
     Bitmap aColorBmp(GetSizePixel(), vcl::PixelFormat::N24_BPP);
     aColorBmp.SetPrefSize(GetPrefSize());
     aColorBmp.SetPrefMapMode(GetPrefMapMode());

Reply via email to