kicad-developers team mailing list archive
-
kicad-developers team
-
Mailing list archive
-
Message #30966
Re: [Patch] pcbnew clipboard support
-
To:
Kristoffer Ödmark <kristofferodmark90@xxxxxxxxx>
-
From:
Tomasz Wlostowski <tomasz.wlostowski@xxxxxxx>
-
Date:
Mon, 25 Sep 2017 11:46:27 +0200
-
Authentication-results:
spf=pass (sender IP is 188.184.36.50) 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;
-
Cc:
Kicad Developers <kicad-developers@xxxxxxxxxxxxxxxxxxx>
-
In-reply-to:
<c85d0659-02eb-0070-6ff5-dd28b6f17020@gmail.com>
-
Spamdiagnosticmetadata:
NSPM
-
Spamdiagnosticoutput:
1:99
-
User-agent:
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.3.0
On 25.09.2017 10:03, Kristoffer Ödmark wrote:
> Maybe I could get access to the work you have done and cherry pick
> patches from there, or just test it and see the difference?
Hi,
Sorry, I totally forgot to send you the link:
https://github.com/twlostow/kicad-dev/tree/tom-copypasta
Tom
>
> - Kristoffer
>
> On 09/25/2017 01:15 AM, Kristoffer Ödmark wrote:
>> On 09/25/2017 12:21 AM, Tomasz Wlostowski wrote:
>>
>>> Hi Kristoffer,
>>>
>>> I've had some time to work on your code during the past few days (I
>>> wasn't aware you're still improving it). I fixed the crashes, refactored
>>> placement of pasted items, fixed the zone refill issue, added snapping
>>> to item's anchors and selection of a reference point.
>>
>> I was not sure that I would find time to improve the code myself.
>> I fixed the crashes, and refactored the placement, and fixed the
>> zone-refill aswell.
>>
>> The adding of a reference point I assume are all on the copy-side of
>> the code, I have not changed that one in a while so that could be
>> merged to either one of our branches
>>>
>>> The only outstanding bug I'm aware of is with net code propagation (not
>>> updated after pasting items).
>> I am not sure what you mean here? I add all the nets that are not
>> existing on the current board and that are used by the imported items,
>> I add them as new nets on the current board ( by creating a
>> NETINFO_ITEM with -1 as netcode and adding it )
>> pcbnew_control.cpp:847-860 in my branch for an example of how. And it
>> seems to work.
>>
>>>
>>> How do you want to proceed with this? I'd like to merge the copy&paste
>>> feature as soon as possible.
>> I would also like the functionality merged as soon as possible and
>> although it is unfortunate that we have now done some duplicate work,
>> it seems you have made some more improvements than me. Merge what you
>> seem fit.
>>
>>>
>>> Best,
>>> Tom
>>>
>
Follow ups
References
-
[Patch] pcbnew clipboard support
From: Kristoffer Ödmark, 2017-05-04
-
Re: [Patch] pcbnew clipboard support
From: Wayne Stambaugh, 2017-06-23
-
Re: [Patch] pcbnew clipboard support
From: Kristoffer Ödmark, 2017-09-17
-
Re: [Patch] pcbnew clipboard support
From: Tomasz Wlostowski, 2017-09-17
-
Re: [Patch] pcbnew clipboard support
From: Kristoffer Ödmark, 2017-09-17
-
Re: [Patch] pcbnew clipboard support
From: Tomasz Wlostowski, 2017-09-17
-
Re: [Patch] pcbnew clipboard support
From: Kristoffer Ödmark, 2017-09-17
-
Re: [Patch] pcbnew clipboard support
From: Kristoffer Ödmark, 2017-09-17
-
Re: [Patch] pcbnew clipboard support
From: Tomasz Wlostowski, 2017-09-18
-
Re: [Patch] pcbnew clipboard support
From: Kristoffer Ödmark, 2017-09-18
-
Re: [Patch] pcbnew clipboard support
From: Tomasz Wlostowski, 2017-09-18
-
Re: [Patch] pcbnew clipboard support
From: Kristoffer Ödmark, 2017-09-18
-
Re: [Patch] pcbnew clipboard support
From: Tomasz Wlostowski, 2017-09-18
-
Re: [Patch] pcbnew clipboard support
From: Kristoffer Ödmark, 2017-09-18
-
Re: [Patch] pcbnew clipboard support
From: Tomasz Wlostowski, 2017-09-18
-
Re: [Patch] pcbnew clipboard support
From: Kristoffer Ödmark, 2017-09-18
-
Re: [Patch] pcbnew clipboard support
From: Tomasz Wlostowski, 2017-09-24
-
Re: [Patch] pcbnew clipboard support
From: Kristoffer Ödmark, 2017-09-24