bug#24260: [PATCH 1/6] dfa: thread-safety: remove 'dfa' global in dfa.c

2016-08-20 Thread Norihiro Tanaka
On Fri, 19 Aug 2016 18:03:19 -0500 Zev Weiss wrote: > Okay -- so your question is about the necessity of making operations other > than dfaexec() thread-safe? That's reasonable, though (obviously) I went > ahead made the other operations thread-safe anyway. > > 1) It was, in some ways, simpl

bug#24259: [PATCH 0/6] dfa: thread safety

2016-08-20 Thread Jim Meyering
On Thu, Aug 18, 2016 at 3:50 AM, Zev Weiss wrote: > Hello, > > Re: Jim Meyering's proposal to move grep's dfa.c into gnulib, I was > hoping I might be able to sneak in the dfa-related parts of my > multithreading patch set before it happens. > > These are all the patches from that set that touch s

bug#24259: [PATCH 0/6] dfa: thread safety

2016-08-20 Thread Zev Weiss
On Sat, Aug 20, 2016 at 05:52:09PM -0700, Jim Meyering wrote: On Thu, Aug 18, 2016 at 3:50 AM, Zev Weiss wrote: Hello, Thank you for separating those so nicely. I have pushed your patches 2..6, now, so will close this. The only changes I made were to the formatting and member ordering of st

bug#24260: [PATCH 1/6] dfa: thread-safety: remove 'dfa' global in dfa.c

2016-08-20 Thread Zev Weiss
On Sun, Aug 21, 2016 at 08:40:59AM +0900, Norihiro Tanaka wrote: On Fri, 19 Aug 2016 18:03:19 -0500 Zev Weiss wrote: Okay -- so your question is about the necessity of making operations other than dfaexec() thread-safe? That's reasonable, though (obviously) I went ahead made the other oper

bug#24259: [PATCH 0/6] dfa: thread safety

2016-08-20 Thread Jim Meyering
On Sat, Aug 20, 2016 at 6:17 PM, Zev Weiss wrote: > On Sat, Aug 20, 2016 at 05:52:09PM -0700, Jim Meyering wrote: >> >> On Thu, Aug 18, 2016 at 3:50 AM, Zev Weiss wrote: >>> >>> Hello, > > >> >> >> Thank you for separating those so nicely. >> I have pushed your patches 2..6, now, so will close t