Subject: Re: PR/33392 CVS commit: src/dist/nawk
To: Aleksey Cheusov <cheusov@tut.by>
From: Christos Zoulas <christos@zoulas.com>
List: netbsd-bugs
Date: 07/26/2006 18:34:19
On Jul 27, 12:23am, cheusov@tut.by (Aleksey Cheusov) wrote:
-- Subject: Re: PR/33392 CVS commit: src/dist/nawk

| You are right, both are correct.
| 
| Suppose
| 1) we are here
|     918         /* add tmpset to current set of states */
|     919         ++(f->curstat);
| +   920         resize_state(f, f->curstat);
|     921         for (i = 0; i < NCHARS; i++)
| 2) 100 states was already created and all tables was properly initialized, i.e.
|    f->state_count == 2.
| 3) f->curstat == 99
| 
| 1) + 2) + 3) ===> unnecessary memory allocation in resize_state(), just a few kbytes.
|   Is there any reason for this?

You need that because gototab[f->curstat] is referenced in 922. I don't
claim to understand the meaning of the variables I am just looking how
they are used.

christos