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: User-Agent: Alpine 2.00 (DEB 1167 2008-08-23) MIME-Version: 1.0 Content-Type: TEXT/PLAIN; format=flowed; charset=US-ASCII 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