[Mono-bugs] [Bug 76176][Nor] Changed - ImageList.AddInternal fails when transforming PixelFormat

[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=76176

--- shadow/76176	2005-09-21 07:29:55.000000000 -0400
+++ shadow/76176.tmp.17046	2005-09-21 08:06:19.000000000 -0400
@@ -2,13 +2,13 @@
 Product: Mono: Class Libraries
 Version: 1.0
 OS: Windows XP (Cygwin)
 OS Details: 
 Status: NEW   
 Resolution: 
-Severity: 
+Severity: Unknown
 Priority: Normal
 Component: Windows.Forms
 AssignedTo: peter at novonyx.com                            
 ReportedBy: jordi at ximian.com               
 QAContact: mono-bugs at ximian.com
 TargetMilestone: ---
@@ -37,6 +37,23 @@
 internally instead of 8 bits.
 
 How to reproducte: add a 24-bitmap into an imagelist. Plataform Mono /
 Win32 / our SD and SWF.
 
 Jordi,
+
+------- Additional Comments From kornelpal at hotmail.com  2005-09-21 08:06 -------
+I think we should use the same solutions on all platforms as far as 
+it is possible, because this makes sure the same behaviour on 
+Mono/Windows and Mono/Linux that is an important thing for people 
+who develop on Windows but target Linux as well as Windows using a 
+single source code and binary.
+
+I don't think that the current solution is bad. Windows ImageList 
+(in comclt32.dll) uses GDI bitmaps (or maybe only a single one) to 
+store images. Our implementation does the same but using GDI+ to be 
+platform independent.
+
+Note that however that the current implementation of this solution 
+is rather poor. For example we should copy all the images when 
+adding and returning a copy of them when getting as mixing things 
+causes inconsistent behaviour.