On 26.03.2015 12:45, jp charras wrote:

> I tested your patches, and they add a *very good and very useful feature*.
> I am willing to commit them, but I'll will *very* happy if you can fix
> some (minor) issues:

I think I've caught most of them.

Are new features still added to the release? I've conveniently ignored
the release preparations so far, and didn't expect to make the cut. :)

> - All dialogs in Kicad have to be internationalized and have a
> RESIZE_BORDER style enabled (an option in wxFormBuilder)

Done, but the column headers are not translated yet (the column objects
are not yet generated by wxFormBuilder).

> - Have a look at our coding style policy: class names (and enums) are in
> uppercase letters.

I think I got it now. I've left the "argument" prefix as "a" instead of
"a_", which is a tiny bit inconsistent, but that's the way everyone does
it, apparently.

   Simon

Attachment: signature.asc
Description: OpenPGP digital signature

_______________________________________________
Mailing list: https://launchpad.net/~kicad-developers
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~kicad-developers
More help   : https://help.launchpad.net/ListHelp

Reply via email to