At 10:44 PM 23/10/2005 -0600, Peter Dennis Bartok wrote:
>Jonathan,
>
>I haven't reviewed the patch yet, but I noticed that you put the tests for 
>it in winforms; it'd be cool you would nunit-ify them and put them straight 
>under System.Drawing/Test? Maybe using GetPixel to verify it rotated 
>correctly. That way it could be automated. The winforms module is mostly 
>meant for visual test applications.
>
>I'll respond on the patch tomorrow, after I had a chance to review.
>
>Cheers,
>  Peter

I poked around in the System.Drawing tests and found that there was already
a test in place for rotation of non-indexed bitmaps, so I added a test for
just 1- and 4-bit bitmaps (see attached patch). It uses Marshal.ReadByte()
in conjunction with Bitmap.LockBits() to read the packed representation of
the pixels. I hope this is okay :-)

Sorry for adding inappropriate tests to the winforms module. If you'd like,
I could hack up the tests there to display their results visually ;-)

Jonathan

Attachment: rotate1bit4bit-test.patch.gz
Description: Binary data

_______________________________________________
Mono-devel-list mailing list
Mono-devel-list@lists.ximian.com
http://lists.ximian.com/mailman/listinfo/mono-devel-list

Reply via email to