Hi jp,

On 13.10.2016 12:42, jp charras wrote:

> Many times, it was said these files must be *never* modified by hand.
> You perfectly know that, and why it was said.

> So, why are doing that?

These patches are not meant to go into the mainline project, just to
demonstrate how I went looking for unused events, so they have to be
human readable (but still apply cleanly).

That's why I split out any large mechanical changes into separate
patches -- when you read patches, just ignore the ones that are only
comment changes, but if you want to test whether the code still works,
they are there so you don't get any errors.

For actual code changes, we'd probably group the changes to several
related dialogs together in a single commit, and readability of the
patch becomes a secondary concern then.

   Simon

Attachment: signature.asc
Description: OpenPGP digital signature

_______________________________________________
Mailing list: https://launchpad.net/~kicad-developers
Post to     : kicad-developers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~kicad-developers
More help   : https://help.launchpad.net/ListHelp

Reply via email to