Subject: Re: PR/33392 CVS commit: src/dist/nawk
To: None <gnats-admin@netbsd.org, netbsd-bugs@netbsd.org,>
From: Dan McMahill <dmcmahill@NetBSD.org>
List: netbsd-bugs
Date: 07/26/2006 04:05:08
The following reply was made to PR bin/33392; it has been noted by GNATS.
From: Dan McMahill <dmcmahill@NetBSD.org>
To: gnats-bugs@NetBSD.org
Cc: Christos Zoulas <christos@zoulas.com>,
Aleksey Cheusov <cheusov@tut.by>
Subject: Re: PR/33392 CVS commit: src/dist/nawk
Date: Tue, 25 Jul 2006 23:51:45 -0400
Dan McMahill wrote:
> The following reply was made to PR bin/33392; it has been noted by GNATS.
>
> From: Dan McMahill <dmcmahill@NetBSD.org>
> To: Christos Zoulas <christos@zoulas.com>
> Cc: Aleksey Cheusov <cheusov@tut.by>, gnats-bugs@NetBSD.org,
> netbsd-bugs@NetBSD.org
> Subject: Re: PR/33392 CVS commit: src/dist/nawk
> Date: Tue, 25 Jul 2006 23:20:41 -0400
>
> Christos Zoulas wrote:
> > On Jul 3, 1:41pm, cheusov@tut.by (Aleksey Cheusov) wrote:
> > -- Subject: Re: PR/33392 CVS commit: src/dist/nawk
> >
> > | Changes you commited to the HEAD related to this PR seems good to me,
> > | everything works correctly and much faster than gawk (for huge
> > | regexps) that i used for years.
> > |
>
> unfortunately, there seem to be some bugs introduced by these commits.
> I'm getting lots of junk pointer complaints from free() followed by a
> segfault. nawk prior to these changes don't have this problem. The
> patch Christos attached to PR/3406 makes the segfaults go away, but this
> new nawk uses more than 10x the RAM on the same script with the same
> input file than an older (prior to these changes) nawk.
>
> I can send the awk script and input file if anyone cares to dig around.
>
I should have read source changes before sending this. Christos, your
1.12 change to src/dist/nawk/b.c seems to have fixes the problems I was
seeing.
Thanks!
-Dan