Re: bug#24256: new module for grep's DFA matcher

2016-09-11 Thread Jim Meyering
On Sat, Sep 10, 2016 at 5:55 PM, Jim Meyering wrote: > On Sat, Sep 10, 2016 at 2:00 PM, Assaf Gordon wrote: >> Hello Jim and all, >> >>> On Sep 10, 2016, at 14:44, Jim Meyering wrote: >>> >>> Finally, here is a similar change, but now for gnu sed. For it, I also >>> had to make some small code c

Re: bug#24256: new module for grep's DFA matcher

2016-09-10 Thread Jim Meyering
On Sat, Sep 10, 2016 at 2:00 PM, Assaf Gordon wrote: > Hello Jim and all, > >> On Sep 10, 2016, at 14:44, Jim Meyering wrote: >> >> Finally, here is a similar change, but now for gnu sed. For it, I also >> had to make some small code changes to adapt to the new >> localeinfo.[ch] and to the dfasy

Re: bug#24256: new module for grep's DFA matcher

2016-09-10 Thread Assaf Gordon
Hello Jim and all, > On Sep 10, 2016, at 14:44, Jim Meyering wrote: > > Finally, here is a similar change, but now for gnu sed. For it, I also > had to make some small code changes to adapt to the new > localeinfo.[ch] and to the dfasyntax API change. I'll wait a day or so > before pushing this,

Re: bug#24256: new module for grep's DFA matcher

2016-08-17 Thread Paul Eggert
This all sounds good, thanks. Your attachment was a submodule diff (basically, a pair of git commit IDs), so not much to review. I have some dfa.c-related changes pending. I assume it'll be OK for me to commit them to grep as needed, and that when you make the switch you'll grab the then-curre