bug#21266: [PATCH] dfa: simplify for non-POSIX locales

2016-07-06 Thread Paul Eggert
Thanks, I merged those two patches and installed them into the grep master, with minor adjustments to the commit messages. I also installed two minor fixup patches, mostly fixing minor style issues. I did notice one minor technical issue; the second patch had code that looked like this: + co

bug#21266: [PATCH] dfa: simplify for non-POSIX locales

2016-07-06 Thread arnold
Hi. Paul Eggert wrote: > Thanks, I merged those two patches and installed them into the grep > master, with minor adjustments to the commit messages. Can you explain the story behind this? Why is support for non-ASCII ranges being pulled out of dfa? I am concerned that this will affect gawk.

bug#21266: [PATCH] dfa: simplify for non-POSIX locales

2016-07-06 Thread Paul Eggert
On 07/06/2016 09:14 PM, arn...@skeeve.com wrote: Can you explain the story behind this? Why is support for non-ASCII ranges being pulled out of dfa? I am concerned that this will affect gawk. This new change shouldn't affect gawk, since that horse left the barn a couple of years ago (grep comm