Re: [PATCH 1/3] dfa: fix dfa-heap-overrun failure

2020-09-15 Thread Paul Eggert
Thanks, I installed your updated dfa.c patch into Gnulib, along with the attached cleanup patch. >From 91e8dccc3c352d56262e24f8fc8d4b736092e639 Mon Sep 17 00:00:00 2001 From: Paul Eggert Date: Tue, 15 Sep 2020 13:44:34 -0700 Subject: [PATCH] dfa: remove dfa-heap-overrun workaround * lib/dfa.c

Re: [PATCH 1/3] dfa: fix dfa-heap-overrun failure

2020-09-15 Thread Norihiro Tanaka
On Tue, 15 Sep 2020 10:43:05 -0700 Jim Meyering 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

Re: [PATCH 1/3] dfa: fix dfa-heap-overrun failure

2020-09-15 Thread Jim Meyering
On Mon, Sep 14, 2020 at 6:29 AM Norihiro Tanaka wrote: > On Mon, 14 Sep 2020 00:28:32 -0700 > Paul Eggert wrote: > > > On 9/14/20 12:13 AM, Norihiro Tanaka wrote: > > > > > when (i >= d->follows[i].elems[j].index), it seems that > > > map[d->follows[i].elems[j].index] has been already set a

Re: Trying to bootstrap my project, distcheck doesn't configure

2020-09-15 Thread Bruce Korb
Hi Bruno, On 9/14/20 3:53 PM, Bruno Haible wrote: Hi Bruce, It would help if you would give a pointer to the source code you are trying to bootstrap. Because a single line in Makefile.am or configure.ac can have a big effect. It lives on my home machine while I try to get it ready for prime