[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 1/3] dfa: fix dfa-heap-overrun failure
From: |
Norihiro Tanaka |
Subject: |
Re: [PATCH 1/3] dfa: fix dfa-heap-overrun failure |
Date: |
Wed, 16 Sep 2020 04:47:08 +0900 |
On Tue, 15 Sep 2020 10:43:05 -0700
Jim Meyering <jim@meyering.net> wrote:
> One nit with the patch: this statement is duplicated:
>
> + position *p = firstpos - stk[-1].nfirstpos;
> + for (position *p = firstpos - stk[-1].nfirstpos;
I removed first declaration.
> And can you clarify your comment below?
>
> > The bug is introduced in commit cafb61533f5bfb989698e3924f97471498b2422b
> > which is a first patch I wrote, and I attach a patch to fix the bug.
>
> That commit refers to a Sept 13 change by Paul Eggert. Is that the
> patch you intended to reference?
Sorry, it points da0e8454a8e68035ef4b87dbb9097f85df6ece27.
Thanks,
Norihiro
0001-dfa-fix-failure-in-removal-of-epsilon-closure.patch
Description: Text document