[Mono-bugs] [Bug 75254][Nor] New - 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 peter at novonyx.com.

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

--- shadow/75254	2005-06-13 12:20:54.000000000 -0400
+++ shadow/75254.tmp.21548	2005-06-13 12:20:54.000000000 -0400
@@ -0,0 +1,33 @@
+Bug#: 75254
+Product: Mono: Class Libraries
+Version: 1.1
+OS: All
+OS Details: 
+Status: NEW   
+Resolution: 
+Severity: 
+Priority: Normal
+Component: Sys.Drawing.
+AssignedTo: mono-bugs at ximian.com                            
+ReportedBy: peter at novonyx.com               
+QAContact: mono-bugs at ximian.com
+TargetMilestone: ---
+URL: 
+Cc: 
+Summary: Palette Patch breaks Icons
+
+When running the winforms/icon test, the following exception gets thrown:
+Unhandled Exception: System.ArgumentException: Invalid Parameter. A null 
+reference or invalid value was found.
+in <0x00063> System.Drawing.GDIPlus:CheckStatus (Status status)
+in <0x0005a> System.Drawing.Bitmap:SetPixel (Int32 x, Int32 y, Color color)
+in (wrapper remoting-invoke-with-check) System.Drawing.Bitmap:SetPixel 
+(int,int,System.Drawing.Color)
+in <0x0048c> System.Drawing.Icon:ToBitmap ()
+
+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.