Subject: Re: CVS commit: basesrc/bin/sh
To: Christos Zoulas <christos@zoulas.com>
From: enami tsugutomo <enami@sm.sony.co.jp>
List: source-changes
Date: 02/14/2002 10:58:02
christos@zoulas.com (Christos Zoulas) writes:

> | >  			varnest++;
> | > +			if (varnest >= maxnest) {
> | > +				maxnest += 32;
> | > +				dblquotep = ckrealloc(dblquotep, maxnest / 8);
> | > +				if (maxnest == 64)
> | > +					*dblquotep = dblquote;
> | 
> | ...here, dblquotep[0] is initialized with dblquote.  It looks
> | inconsistient.
> 
> Yes, but I wanted to keep the dblquotep array correct no matter if it is used
> or not. It is a good programming practice to keep data consistent.

Once the varnest become < 32 again, data will be inconsistient.

enami.