[Mono-bugs] [Bug 75254][Nor] Changed - Palette Patch breaks Icons

[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 4lw0e0402 at sneakemail.com.

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

--- shadow/75254	2005-06-13 12:20:54.000000000 -0400
+++ shadow/75254.tmp.25015	2005-06-13 16:01:07.000000000 -0400
@@ -1,14 +1,14 @@
 Bug#: 75254
 Product: Mono: Class Libraries
 Version: 1.1
 OS: All
 OS Details: 
-Status: NEW   
-Resolution: 
-Severity: 
+Status: RESOLVED   
+Resolution: NOTABUG
+Severity: Unknown
 Priority: Normal
 Component: Sys.Drawing.
 AssignedTo: mono-bugs at ximian.com                            
 ReportedBy: peter at novonyx.com               
 QAContact: mono-bugs at ximian.com
 TargetMilestone: ---
@@ -28,6 +28,14 @@
 The problem is this line
         if (gdip_is_an_indexed_pixelformat (data->PixelFormat))
                 return InvalidParameter;
 in GdipBitmapSetPixel, it simply returns an InvalidParameter, instead of 
 making the pixel transparent. This worked before the palette patch was 
 added.
+
+------- Additional Comments From 4lw0e0402 at sneakemail.com  2005-06-13 16:01 -------
+This is actually a bug in the code that is trying to use SetPixel on 
+an indexed image. It worked before because internally the image was 
+being upsampled to 32-bit ARGB at load time. The fix/workaround is to 
+modify the code that's calling SetPixel to convert the pixel format 
+to 32-bit ARGB manually, after which SetPixel will behave as it did 
+before.