Removed a really unpleasant test for whether a pointer points at a string
constant or not by comparing it with 'end', which presumably points at the top of the BSS. In fact, on Linux it evaluates to NULL, and on OSX it doesn't exist at all, so it appears to be unnecessary.
This commit is contained in:
@@ -281,15 +281,13 @@ setallreg(vi) struct varinfo *vi; {
|
||||
|
||||
freevi(vip) register struct varinfo *vip; {
|
||||
register i;
|
||||
extern char *end;
|
||||
|
||||
if (vip==0)
|
||||
return;
|
||||
freevi(vip->vi_next);
|
||||
freevi(vip->vi_vi);
|
||||
for (i=0;i<VI_NSTR;i++)
|
||||
if (vip->vi_str[i]>end)
|
||||
free((char *) vip->vi_str[i]);
|
||||
free((char *) vip->vi_str[i]);
|
||||
free(vip);
|
||||
}
|
||||
|
||||
|
||||
Reference in New Issue
Block a user