On Fri, Nov 25, 2016 at 11:03 AM, Paul Eggert <egg...@cs.ucla.edu> wrote: > Norihiro Tanaka wrote: >> >> Can anyone review this change? > > > Thanks for doing all that, and sorry about the late review. I reviewed it, > tweaked its commit message and propagated that into ChangeLog (the gnulib > practice), and installed the result into gnulib, with two followup patches > that I hope are self-explanatory. All three patches are attached. Please let > us know of any problems you see with the result. > > CC'ing this to grep-devel since that's where the hardy band of dfa consumers > hang out.
Thanks to both of you. I confirmed that with those, grep still passes all of its tests, even with ASAN (and hence leak detection) enabled. Paul, I see one typo in your 2nd change's log: s/cdalls/calls/ Omit unnecessary cdalls to zeroset.