Need help on couple of patches

Juan Lang juan.lang at gmail.com
Wed Feb 18 12:32:16 CST 2009


Hi Igor,

> It's a single-line patch that adds forwarding notification that was
> omitted in wine code. This fix looks pretty simple, but it included
> lots of reading/testing/debugging, and here is what I came with. And
> as I told there it fixes one wine bugzilla bug.

A regression test would go a long way toward proving your fix is correct.

> The patch adresses 3 problems that all together fix that bug. My
> question there was if I should send it in one patch or in several. But
> it seems that no one have noticed that post.

Several patches would definitely improve this one's chance of success.

Getting your first patch committed is usually the toughest.  Good
luck, and thanks for helping Wine,
--Juan



More information about the wine-devel mailing list