Re: What's wrong with my patch?

[Andrew Eikum]

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