What's wrong with my patch?
Andrew Eikum
aeikum at codeweavers.com
Mon Aug 31 11:28:29 CDT 2009
Jan de Mooij wrote:
> Hello.
>
> Subject says it all. I suppose the tests I added may fail on other
> locales, but there is no other way to test my patch?
> Should i send it without the tests?
>
> Thanks in advance.
>
> Jan
I can't speak much to the patch itself, but the patch file you attached
has an invalid email address. You should set user.name and user.email
in .git/config. See `man git-config` for more info.
Also, #defining the ID for the apply button without #undefing it seems
wrong. I'd use a const INT in the test_buttons method so the scope is
taken care of automatically.
Andrew
More information about the wine-devel
mailing list