[Mono-bugs] [Bug 76562][Blo] Changed - 32-bit BMPs should be loaded as Format32bppRgb

[bugzilla-daemon at bugzilla.ximian.com]

Please do not reply to this email- if you want to comment on the bug, go to the
URL shown below and enter your comments there.

Changed by kornelpal at hotmail.com.

http://bugzilla.ximian.com/show_bug.cgi?id=76562

--- shadow/76562	2005-10-26 10:48:24.000000000 -0400
+++ shadow/76562.tmp.21690	2005-10-28 08:05:45.000000000 -0400
@@ -2,14 +2,14 @@
 Product: Mono: Class Libraries
 Version: unspecified
 OS: All
 OS Details: 
 Status: NEW   
 Resolution: 
-Severity: 
-Priority: Normal
+Severity: Unknown
+Priority: Blocker
 Component: libgdiplus
 AssignedTo: jordi at ximian.com                            
 ReportedBy: kornelpal at hotmail.com               
 QAContact: mono-bugs at ximian.com
 TargetMilestone: ---
 URL: 
@@ -22,6 +22,26 @@
 alpha channel for Format32bppRgb but alpha channel is preserved in memory 
 and can be accessed using LockBits even when locking with Format32bppRgb. 
 (I think locking in format other than the format of bitmap is not 
 supported by MS GDI+.)
 
 libgdiplus should do the same.
+
+------- Additional Comments From kornelpal at hotmail.com  2005-10-28 08:05 -------
+The following patch seems to fix the bug without side effects.
+
+Please approve or commit it as this patch is required for 
+ImageListStreamer.
+
+Index: bmpcodec.c
+===================================================================
+--- bmpcodec.c	(revision 52314)
++++ bmpcodec.c	(working copy)
+@@ -76,7 +76,7 @@
+ {
+         switch (bitcount) {
+         case 32:
+-                return Format32bppArgb;
++                return Format32bppRgb;
+         case 24:
+                 return Format24bppRgb;
+         case 8: