Modified the "gds write" command so that it issues a strongly-
worded warning if an abstract cell view is written to GDS. Corrected the "cellname ... writeable" command to allow an overrride of the read-only status of a cell. That change had been made before but apparently got reverted by the recent rollback.
This commit is contained in:
parent
04fca92bf0
commit
521baa91d9
|
|
@ -814,6 +814,13 @@ calmaProcessDef(def, outf, do_library)
|
||||||
|
|
||||||
if (isReadOnly && hasContent && CalmaAddendum) return (0);
|
if (isReadOnly && hasContent && CalmaAddendum) return (0);
|
||||||
|
|
||||||
|
/* Give a strongly-worded statement about writing abstract views */
|
||||||
|
|
||||||
|
if (isAbstract && !isReadOnly)
|
||||||
|
TxError("Warning: Writing abstract view of \"%s\" to GDS. This is"
|
||||||
|
" probably not what you want to do.\n",
|
||||||
|
def->cd_name);
|
||||||
|
|
||||||
/*
|
/*
|
||||||
* Output the definitions for any of our descendants that have
|
* Output the definitions for any of our descendants that have
|
||||||
* not already been output. Numbers are assigned to the subcells
|
* not already been output. Numbers are assigned to the subcells
|
||||||
|
|
|
||||||
|
|
@ -1084,13 +1084,11 @@ CmdCellname(w, cmd)
|
||||||
if (cellDef->cd_fd == -1)
|
if (cellDef->cd_fd == -1)
|
||||||
dbReadOpen(cellDef, NULL, TRUE, NULL);
|
dbReadOpen(cellDef, NULL, TRUE, NULL);
|
||||||
|
|
||||||
if (cellDef->cd_fd != -1)
|
if (cellDef->cd_fd == -1)
|
||||||
cellDef->cd_flags &= ~CDNOEDIT;
|
TxError("Overriding advisory lock held on cell %s\n",
|
||||||
else
|
cellDef->cd_name);
|
||||||
TxError("Advisory lock held on cell %s\n", cellDef->cd_name);
|
|
||||||
#else
|
|
||||||
cellDef->cd_flags &= ~CDNOEDIT;
|
|
||||||
#endif
|
#endif
|
||||||
|
cellDef->cd_flags &= ~CDNOEDIT;
|
||||||
WindAreaChanged(w, &w->w_screenArea);
|
WindAreaChanged(w, &w->w_screenArea);
|
||||||
CmdSetWindCaption(EditCellUse, EditRootDef);
|
CmdSetWindCaption(EditCellUse, EditRootDef);
|
||||||
}
|
}
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue