← Back to team overview

kicad-developers team mailing list archive

Re: [PATCH] Add pin overview table

 

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


Follow ups

References