luke
/
magic
mirror of https://github.com/RTimothyEdwards/magic.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Extended the "cifmaxwidth" function to include the option "both", 

which has the same meaning as the "maxwidth" function ("both"
checks either tile dimension to see if it exceeds the maximum).
This is a simple per-tile check and assumes that violations do
not occur across multiple tiles.  This should be sufficient for
most checks.
This commit is contained in:
Tim Edwards 2024-12-08 12:14:38 -05:00
parent dc87a8c693
commit 37dfe07edf
2 changed files with 36 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
VERSION
View File

@ -1 +1 @@
8.3.504 8.3.505

42
drc/DRCcif.c
View File

@ -1268,11 +1268,12 @@ drcCifMaxwidth(argc, argv)
return (0); return (0);
} }
if (strcmp(bends,"bend_illegal") == 0) bend =0; if (strcmp(bends, "bend_illegal") == 0) bend = 0;
else if (strcmp(bends,"bend_ok") == 0) bend =DRC_BENDS; else if (strcmp(bends, "bend_ok") == 0) bend = DRC_BENDS;
else if (strcmp(bends, "both") == 0) bend = DRC_MAXWIDTH_BOTH;
else else
{ {
TechError("unknown bend option %s\n",bends); TechError("unknown bend option %s\n", bends);
return (0); return (0);
} }
@ -1447,10 +1448,35 @@ drcCheckCifMaxwidth(starttile,arg,cptr)
if (DRCstack == (Stack *) NULL) if (DRCstack == (Stack *) NULL)
DRCstack = StackNew(64); DRCstack = StackNew(64);
/* if bends are allowed, just check on a tile-by-tile basis that /* If bends are allowed, just check on a tile-by-tile basis that
one dimension is the max. This is pretty stupid, but it correctly * both dimensions exceed the max. This is pretty stupid, but it
calculates the trench width rule. dcs 12.06.89 */ * correctly calculates, e.g., the trench width rule. dcs 12.06.89
if (cptr->drcc_flags & DRC_BENDS) *
* Do the same for the "both" case, checking on a tile-by-tile basis
* that either dimension exceeds the max.
*/
if (cptr->drcc_flags & DRC_MAXWIDTH_BOTH)
{
Rect rect;
TiToRect(starttile,&rect);
if ((rect.r_xtop-rect.r_xbot > edgelimit) ||
(rect.r_ytop-rect.r_ybot > edgelimit))
{
rect.r_xbot /= scale;
rect.r_xtop /= scale;
rect.r_ybot /= scale;
rect.r_ytop /= scale;
GeoClip(&rect, arg->dCD_clip);
if (!GEO_RECTNULL(&rect)) {
(*(arg->dCD_function)) (arg->dCD_celldef, &rect,
arg->dCD_cptr, arg->dCD_clientData);
(*(arg->dCD_errors))++;
}
}
return;
}
else if (cptr->drcc_flags & DRC_BENDS)
{ {
Rect rect; Rect rect;
TiToRect(starttile,&rect); TiToRect(starttile,&rect);
@ -1470,7 +1496,9 @@ drcCheckCifMaxwidth(starttile,arg,cptr)
} }
return; return;
} }
/* Mark this tile as pending and push it */ /* Mark this tile as pending and push it */
PUSHTILE(starttile); PUSHTILE(starttile);
TiToRect(starttile,&boundrect); TiToRect(starttile,&boundrect);