X-Authentication-Warning:	delorie.com: mail set sender to geda-user-bounces using -f
X-Recipient:	geda-user AT delorie DOT com
Date:	Sat, 28 Jan 2017 14:52:58 +0100 (CET)
X-X-Sender:	igor2 AT igor2priv
To:	geda-user AT delorie DOT com
X-Debug:	to=geda-user AT delorie DOT com from="gedau AT igor2 DOT repo DOT hu"
From:	gedau AT igor2 DOT repo DOT hu
Subject:	[geda-user] [pcb] layer delete memleak + bugreport
Message-ID:	<alpine.DEB.2.00.1701281442360.7286@igor2priv>
User-Agent:	Alpine 2.00 (DEB 1167 2008-08-23)
MIME-Version:	1.0
Reply-To:	geda-user AT delorie DOT com

Hi all,

I don't get this part of the code: MoveLayer(), new_index == -1. It does 
memmove()'s to get rid of the layer in the layer array, but I don't see 
any code that would free the layer's object arrays or rtrees.

Are those all just memleaked?

Related bug:

1. put two layers in the same group

2. draw a big rectangle on one of the layers and a line crossing it on the 
other; let it clear the poly

3. delete the layer with the line

4. the line is gone, as expected; because there's no object removal on 
layer removal, the polygon still has the clearance cutout for the line 
until the poly gets clipped for some other reason. Simply noone told the 
poly to reclip because the line is not really gone, it's just 
unaccessible.

Regards,

Igor2

- Raw text -