From 33edee6b425be2bc8aa9999a3820e802126480c7 Mon Sep 17 00:00:00 2001 From: Tim Edwards Date: Wed, 19 May 2021 17:21:59 -0400 Subject: [PATCH] Modified the condition under which the (UNNAMED) file is deleted on loading a layout to avoid doing so with in a suspendall ... resumeall block. That avoids weird errors occurring when the PDK toolkit scripts are run to generate a new device layout if the top level layout is still (UNNAMED) and empty. --- VERSION | 2 +- dbwind/DBWprocs.c | 8 +++++++- 2 files changed, 8 insertions(+), 2 deletions(-) diff --git a/VERSION b/VERSION index 7d3253ea..8b702771 100644 --- a/VERSION +++ b/VERSION @@ -1 +1 @@ -8.3.163 +8.3.164 diff --git a/dbwind/DBWprocs.c b/dbwind/DBWprocs.c index e1c05368..c9d998a7 100644 --- a/dbwind/DBWprocs.c +++ b/dbwind/DBWprocs.c @@ -549,8 +549,14 @@ DBWloadWindow(window, name, flags) /* If the cell before loading was (UNNAMED) and it was */ /* never modified, then delete it now. */ + /* Caveat: The (UNNAMED) cell could be on a push stack; */ + /* that is not fatal but should be avoided. Since the most */ + /* common use is from the toolkit scripts, then make sure */ + /* this doesn't happen within suspendall ... resumeall. */ + if (deleteDef != NULL) - DBCellDelete(deleteDef->cd_name, TRUE); + if (GrDisplayStatus != DISPLAY_SUSPEND) + DBCellDelete(deleteDef->cd_name, TRUE); } /* This function is called for each cell whose expansion status changed.