_NO_CHANGE is -1, so x will always be greater, and a separate test is

not needed. Which makes it asymmetrical; and to depend on the known
value of a define defeats the purpose of using a define; so I'd test it
anyway, but this is a critical performance section.
This commit is contained in:
William McBrine
2006-01-18 19:40:08 +00:00
parent 54a90a9851
commit 64a00d3759

View File

@@ -41,7 +41,7 @@
#endif
#ifdef PDCDEBUG
char *rcsid_addchstr = "$Id: addchstr.c,v 1.12 2006/01/18 19:30:22 wmcbrine Exp $";
char *rcsid_addchstr = "$Id: addchstr.c,v 1.13 2006/01/18 19:40:08 wmcbrine Exp $";
#endif
/*man-start*********************************************************************
@@ -174,7 +174,7 @@ int n;
if (x < minx || minx == _NO_CHANGE)
minx = x;
if (x > maxx || maxx == _NO_CHANGE)
if (x > maxx)
maxx = x;
PDC_LOG(("y %d x %d minx %d maxx %d *ptr %x *ch"