Search and replace: Delete instances crashes KLayout

KLayout / klayout

KLayout Main Sources

http://www.klayout.org

GNU General Public License v3.0

754 stars 196 forks source link

Search and replace: Delete instances crashes KLayout #1671

Closed klayoutmatthias closed 3 months ago

klayoutmatthias commented 3 months ago

This refers to https://www.klayout.de/forum/discussion/2478

A test case is available.

klayoutmatthias commented 3 months ago

The problem is in the ambiguity between instances from cells from a cell tree (the instances inside the child cells) and the instance-expanded tree (also "instances of"). With "delete instances of TOP...*" the child cells are visited multiple times as "instances of ..." denotes the instance-expanded tree and at the second visit the error is thrown.