kicad-developers team mailing list archive
-
kicad-developers team
-
Mailing list archive
-
Message #27438
Re: [PATCH] Add exchange footprints to GAL
On Tue, Jan 31, 2017 at 10:45:42AM -0500, Wayne Stambaugh wrote:
> On 1/31/2017 10:36 AM, Chris Pavlina wrote:
> > On Tue, Jan 31, 2017 at 11:32:40PM +0800, John Beard wrote:
> >> Hi,
> >>
> >> The attached patches add the "exchange footprints" command to the GAL edit tool.
> >>
> >> This is 1 out of 3 parts of
> >> https://bugs.launchpad.net/kicad/+bug/1541460. I'm not sure how to put
> >> this as a commit comment exactly.
> >
> > Generally I close the bug with a comment requesting the submitter open
> > several new bugs for each part of the combined report.
> >
> > *glares at Wayne*
> >
> > ;)
>
> *Wayne glares back* ;) I never get tired of asking people to file
> single issue bug reports when it's right there in the bug reporting page
> on launchpad under the heading "KiCad bug reporting guidelines:". Sigh!
> Maybe it needs to be in a bigger bold font in all capital letters?
Lol Wayne, _you_ filed this report!
>
> >
> >>
> >> This is implemented as two segments:
> >>
> >> * Put the dialog/exchange FP code into pcbnew/dialogs directory (as a
> >> header/implementation pair) where it can be reached by bother
> >> pcbframe.cpp and GAL tools. This also makes it easier to remove legacy
> >> calling code in future. No change is made to the functionality of the
> >> dialog.
> >> * Add the tool to the GAL, calling the dialog as needed.
> >>
> >> The third patch is optional, but tidies up a copy-pasted selection
> >> routine into a named function and adds a little commentary to a couple
> >> of related existing functions.
> >>
> >> Cheers,
> >>
> >> John
> >
> >
> > _______________________________________________
> > Mailing list: https://launchpad.net/~kicad-developers
> > Post to : kicad-developers@xxxxxxxxxxxxxxxxxxx
> > Unsubscribe : https://launchpad.net/~kicad-developers
> > More help : https://help.launchpad.net/ListHelp
> >
Follow ups
References