X-Authentication-Warning: delorie.com: mail set sender to geda-user-bounces using -f
X-Recipient: geda-user AT delorie DOT com
X-Original-DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;
	d=s5tehnika.net; s=default; h=Content-Transfer-Encoding:Content-Type:
	In-Reply-To:MIME-Version:Date:Message-ID:From:References:To:Subject:Sender:
	Reply-To:Cc:Content-ID:Content-Description:Resent-Date:Resent-From:
	Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Id:List-Help:
	List-Unsubscribe:List-Subscribe:List-Post:List-Owner:List-Archive;
	bh=hxm0bbLdNTVFl1E9YRZWSmfjnw8PUz5iexEShZQhPlM=; b=pXPNdi5U9/XV4YYMcGI3492vwM
	q71/W1bMpVf7G4ljHpmMDLamrMkhB9yW9xIGRNYax76DAeEPh6OtrDFTC9YI2Qh36ZnURqVJ+Cq05
	+OSy3NgTlO+MKsLBwQISg1uCEKqKkG1JCOMis/1t6TMwtks4WmMTtSBriWN9Zl7pNk5lLAs2vHvmV
	dcPb5fXseay8gUV6ZIo6S7qo3T+MZfCHQyto59jzT7GmlGGsCAl6/95VO3JHRDdu0lpeBOqEUerkA
	7UR9aN6t7j1uo5PIh9iruUtnGhIl68rDpRzaBT0cw/79RfejC+NIz1lXoB/q3NxmHfcUUQnRR+S3L
	XS1GOtJw==;
Subject: Re: [geda-user] pcb: polygon "twin hole" bug - fixed
To: geda-user AT delorie DOT com
References: <alpine DOT DEB DOT 2 DOT 00 DOT 1607070651120 DOT 7286 AT igor2priv>
 <4f9dfd62-45cd-a307-a6d9-93b7cd4333a2 AT gmail DOT com>
 <alpine DOT DEB DOT 2 DOT 00 DOT 1607241855150 DOT 7286 AT igor2priv>
 <f9211aa4-f13e-8ce6-171b-9abf3e5faab0 AT s5tehnika DOT net>
 <1efdee06-c218-7cac-4029-4f42e420ec82 AT s5tehnika DOT net>
From: "Brane2 (brane2 AT s5tehnika DOT net) [via geda-user AT delorie DOT com]" <geda-user AT delorie DOT com>
Message-ID: <582b39fe-fc83-7b80-8e9c-95459f5b8b1e@s5tehnika.net>
Date: Mon, 25 Jul 2016 02:54:24 +0200
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101
 Thunderbird/45.2.0
MIME-Version: 1.0
In-Reply-To: <1efdee06-c218-7cac-4029-4f42e420ec82@s5tehnika.net>
Content-Type: text/plain; charset=windows-1252; format=flowed
Content-Transfer-Encoding: 7bit
X-AntiAbuse: This header was added to track abuse, please include it with any abuse report
X-AntiAbuse: Primary Hostname - server.studiofaca.com
X-AntiAbuse: Original Domain - delorie.com
X-AntiAbuse: Originator/Caller UID/GID - [47 12] / [47 12]
X-AntiAbuse: Sender Address Domain - s5tehnika.net
X-Get-Message-Sender-Via: server.studiofaca.com: authenticated_id: brane2 AT s5tehnika DOT net
X-Authenticated-Sender: server.studiofaca.com: brane2 AT s5tehnika DOT net
Reply-To: geda-user AT delorie DOT com
Errors-To: nobody AT delorie DOT com
X-Mailing-List: geda-user AT delorie DOT com
X-Unsubscribes-To: listserv AT delorie DOT com
Precedence: bulk

OK. I applied it to official pcb-git and it sorts of works.

I don't need separate first click for polygone selection now, but 
polygon selection doesn't work now as advertised.
I can draw a hole now in a polygon even if my first clicks misses all 
polygones. Hole will stick to the polygon that I happen to nab first 
when clicking on further hole vertexes.
Also, If i start drawing hole over several polygons that partially 
cover, like top and bottom layer, for some reason hole will stick to 
polygon on top layer, even if I happen to be on the bottom layer at that 
time.



On 25. 07. 2016 02:19, Brane2 (brane2 AT s5tehnika DOT net) [via 
geda-user AT delorie DOT com] wrote:
> Ahh, now I understand, deafening silence on the matter - this pcb-rnd 
> is actually separate, svn repo.
>
> Someone _could_ have pointed that out.
>
> Anyway, I'll try to apply change to my copy of official repo and see 
> if it reacts sanely...
>
>
>
>
>
>
>
> On 24. 07. 2016 21:47, Brane2 (brane2 AT s5tehnika DOT net) [via 
> geda-user AT delorie DOT com] wrote:
>> I git log-ed  pcb-rnd branch, but can't find in it any commit that 
>> was more recent than 2013. ( repo as listed on official page for pcb)
>>
>> What am I missing ?
>>
>>> Thanks everyone who has tested this and special thanks to Evan for 
>>> the detailed bug description. The fact that changing the tool 
>>> between the two holes made it very easy to find the bug: it was just 
>>> one of the crosshair states not properly reset, so the second hole 
>>> didn't start in the init step but in a mid-point-step.
>>>
>>> Fixed in pcb-rnd r2308. Trivial one-line diff, should be applied to 
>>> mainline. Anyone testing pcb-rnd, please let me know if poly holes 
>>> break any other way.
>>>
>>> Regards,
>>>
>>>
>>>
>>> On 24. 07. 2016 18:58, gedau AT igor2 DOT repo DOT hu wrote:
>>>
>>> Igor2
>>>
>>
>>
>
>