[Mono-bugs] [Bug 78184][Nor] Changed - ErrorProvider rendering in MWF is broken

[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 xenomorph2 at onlinehome.de.

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

--- shadow/78184	2006-04-25 06:56:37.000000000 -0400
+++ shadow/78184.tmp.9520	2006-05-03 09:11:49.000000000 -0400
@@ -41,6 +41,20 @@
 screen capture showing the problem
 
 
 ------- Additional Comments From alextxm at tin.it  2006-04-25 06:56 -------
 forgot to mention: tested with mono 1.1.12/13.x/14/15 (screen capture
 is with mono 1.1.15 (beta #2))
+
+------- Additional Comments From xenomorph2 at onlinehome.de  2006-05-03 09:11 -------
+This happens because the errorprovider icon is 24Bit. That means, the
+icon data that is stored in IconImage.iconXOR doesn't use 4 bytes to
+store the value of each color instead it uses only 3 bytes.
+
+So, using Marshal.Copy to transfer RGB values from IconImage.iconXOR
+to ARGB bitmap memory doesn't work. Instead we have to read 3 bytes
+from iconXOR and then set the color for the pixel in the bitmap that
+is created in Icon.ToBitmap(), read the next 3 bytes..., and so on.
+
+Attached is a patch that fixes it.
+If bug #78189 (Bmpcodec needs to be fixed to also support 24bpp) gets
+fixed this can be reverted again !?! Am I right ?