kicad-developers team mailing list archive
-
kicad-developers team
-
Mailing list archive
-
Message #27634
Re: Pad import/export/push and modedit mirror
-
To:
<kicad-developers@xxxxxxxxxxxxxxxxxxx>
-
From:
Maciej Suminski <maciej.suminski@xxxxxxx>
-
Date:
Tue, 7 Feb 2017 23:24:51 +0100
-
Authentication-results:
spf=pass (sender IP is 188.184.36.48) smtp.mailfrom=cern.ch; lists.launchpad.net; dkim=none (message not signed) header.d=none;lists.launchpad.net; dmarc=bestguesspass action=none header.from=cern.ch;
-
In-reply-to:
<CAOuK9LgNANcEagLCoE46PJUSd=csR8nBgHiYh7kHqfiVv-sc_A@mail.gmail.com>
-
Spamdiagnosticmetadata:
NSPM
-
Spamdiagnosticoutput:
1:99
-
User-agent:
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.6.0
Hi Nick,
Thank you for the patches, I have just pushed them. I squeezed the two
patches that rename and regenerate bitmap data, I think they should go
together.
Regards,
Orson
On 02/04/2017 01:10 PM, Nick Østergaard wrote:
> I have attached a patch that renames this. It contains three commits
> for easier review.
>
> You should be able to apply with git am rename_pad_settings_tool.patch.
>
> 2017-01-25 19:13 GMT+01:00 Nick Østergaard <oe.nick@xxxxxxxxx>:
>> I have always had the feeling that import and export was a bit strange
>> user of terminology. I do find it hard to find some better words, but
>> what about copy and apply?
>>
>> Import becomes apply and export becomes copy?
>>
>> (and then swap the order of them in the list for a more logical ordering)
>>
>> 2017-01-25 15:35 GMT+01:00 Wayne Stambaugh <stambaughw@xxxxxxxxx>:
>>> @Orson, would you please take a look at this patch when you get a
>>> chance. They both look good to me.
>>>
>>> I did see a few coding policy issues. Mostly this:
>>>
>>> + if ( selection[0]->Type() == PCB_PAD_T )
>>>
>>> which should be:
>>>
>>> + if( selection[0]->Type() == PCB_PAD_T )
>>>
>>> @John, please clean these up and resubmit the patches when you get a chance.
>>>
>>> Thanks,
>>>
>>> Wayne
>>>
>>> On 1/25/2017 2:39 AM, John Beard wrote:
>>>> Sorry, attached the wrong version of the "import export" patch - this
>>>> is the correct sequence.
>>>>
>>>> On Wed, Jan 25, 2017 at 12:41 AM, John Beard <john.j.beard@xxxxxxxxx> wrote:
>>>>> Hi,
>>>>>
>>>>> Here are two patches for outstanding GAL functionality: pad
>>>>> import/export/push and modedit mirroring.
>>>>>
>>>>> Pad "push" is what I termed global edit, since I think that describes
>>>>> the action of taking a reference and "pushing" it to a filtered subset
>>>>> of other pads. I am happy to keep the old name too, if that's
>>>>> preferred.
>>>>>
>>>>> I have implemented in a new PAD_TOOL GAL tool as I think there could
>>>>> be more pad-only function in future that can all stick together in
>>>>> there.
>>>>>
>>>>> Let me know if there are any suggestions!
>>>>>
>>>>> 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
>>>
>>> _______________________________________________
>>> 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
>>>
>>>
>>> _______________________________________________
>>> 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