Added GDI+ public headers, small implementation of pen, memory and initialization, and two tests (2nd try)

Detlef Riekenberg wine.dev at web.de
Fri Jun 1 08:19:53 CDT 2007


On Do, 2007-05-31 at 18:02 -0700, Evan Stade wrote:

> Changelog:
> *Added GDI+ public headers
> *Added some implementation (pen.c, gdiplus.c, memory.c)
> *Added two tests (pen.c, init.c)

I good indicator, that the Patch should be split in smaller Parts,
is a Changelog (Subject) that is larger than a simple Line 
or includes an enumeration.

You already use 3 Lines in the Changelog which describe the code for
a minimum of 3 Patches.
The usual way is one Patch for every new header to reduce the
complexity of the Patch, but your headers are really small, 
so 2 Patches or maybe 1 single header-Patch might be ok
(this depend also on the size).


Thanks

-- 
 
By by ... Detlef





More information about the wine-devel mailing list