[advpack/setupapi] LaunchINFSection

Raphael fenix at club-internet.fr
Sun Nov 20 19:25:24 CST 2005


Hi,

> I appreciate your work on advpack, but some comments that might help
> get it accepted more easily:
>
> * separate the patch into as many independent chunks as you can.  It
> seems like each function you implement can be sent in separate and
> linearly.

Yes, 
but as they are really small changes i keep on patch for that

> * it would help to add test cases wherever you can even if it's just
> to check how the function handles parameters.

I have started to think about that but need a complex test to test all 
capabilities (ie other than parse of command line)
(see answer to Markus)

> > +    if (!bRet) {
> >
> > +      ERR("Failed to CopyFileW(%s, %s)\n", debugstr_w(src),
> > debugstr_w(dst));
> >
> > +    }
>
> The rest of the file uses standard C indentation, so it's best to
> stick with that:

forget to change that, good catch :)

> --
> James Hawkins

Regards,
Raphael
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
Url : http://www.winehq.org/pipermail/wine-devel/attachments/20051121/5326f060/attachment.pgp


More information about the wine-devel mailing list