Re: [hackers] [sbase][PATCH] grep: remove = flag from readme

2017-01-02 Thread Laslo Hunhold
On Tue, 27 Dec 2016 18:32:52 +0100 Mattias Andrée wrote: Hey Mattias, > We still need a new regex-engine to support NUL bytes, > but perhaps that can be circumvented. We also need a > faster engine, currently both musl and glibc is too > slow for any serious grepping. agreed.

Re: [hackers] [sbase][PATCH] grep: remove = flag from readme

2016-12-27 Thread Mattias Andrée
On Tue, 27 Dec 2016 09:26:34 -0800 Evan Gates wrote: > On Tue, Dec 27, 2016 at 3:09 AM, Laslo Hunhold > wrote: > > On Wed, 30 Mar 2016 19:01:16 +0200 > > Mattias Andrée wrote: > > > > Hey Mattias, > > > >> $ echo äö | ./grep [å] > >>

Re: [hackers] [sbase][PATCH] grep: remove = flag from readme

2016-12-27 Thread Evan Gates
On Tue, Dec 27, 2016 at 3:09 AM, Laslo Hunhold wrote: > On Wed, 30 Mar 2016 19:01:16 +0200 > Mattias Andrée wrote: > > Hey Mattias, > >> $ echo äö | ./grep [å] >> äö >> >> This is not want one expects from >> a program that supports UTF-8. > > as a general

Re: [hackers] [sbase][PATCH] grep: remove = flag from readme

2016-12-27 Thread Laslo Hunhold
On Wed, 30 Mar 2016 19:01:16 +0200 Mattias Andrée wrote: Hey Mattias, > $ echo äö | ./grep [å] > äö > > This is not want one expects from > a program that supports UTF-8. as a general note, we may think about adding a setlocale() when we access the regex-engine. What do

Re: [hackers] [sbase][PATCH] grep: remove = flag from readme

2016-04-03 Thread Mattias Andrée
On Sun, 3 Apr 2016 14:55:35 -0700 Eric Pruitt wrote: > On Sun, Apr 03, 2016 at 11:50:08PM +0200, Mattias Andrée > wrote: > > That works with musl but not glibc. > > If you use 'setlocale(LC_ALL, "")' with GCC, grep behaves > as expected: I assume you mean glibc. Does

Re: [hackers] [sbase][PATCH] grep: remove = flag from readme

2016-04-03 Thread Eric Pruitt
On Sun, Apr 03, 2016 at 11:50:08PM +0200, Mattias Andrée wrote: > That works with musl but not glibc. If you use 'setlocale(LC_ALL, "")' with GCC, grep behaves as expected: sbase$ echo äö | ./grep '[å]' (1) sbase$ echo å | ./grep '[å]' å Eric

Re: [hackers] [sbase][PATCH] grep: remove = flag from readme

2016-04-03 Thread Mattias Andrée
That works with musl but not glibc. On Sun, 3 Apr 2016 23:39:38 +0200 FRIGN wrote: > On Sun, 3 Apr 2016 23:35:21 +0200 > Mattias Andrée wrote: > > Hey Mattias, > > > Perhaps it could be noted. But it fails > > with both musl and glibc. I think we need new > >

Re: [hackers] [sbase][PATCH] grep: remove = flag from readme

2016-04-03 Thread FRIGN
On Sun, 3 Apr 2016 23:35:21 +0200 Mattias Andrée wrote: Hey Mattias, > Perhaps it could be noted. But it fails > with both musl and glibc. I think we need new > regex engine, perhaps adapted from musl, > that supports NUL bytes and UTF-8 transparently. you could also try a

Re: [hackers] [sbase][PATCH] grep: remove = flag from readme

2016-04-03 Thread Mattias Andrée
Perhaps it could be noted. But it fails with both musl and glibc. I think we need new regex engine, perhaps adapted from musl, that supports NUL bytes and UTF-8 transparently. maandree On Sun, 3 Apr 2016 22:30:45 +0200 FRIGN wrote: > On Wed, 30 Mar 2016 19:01:16 +0200 > Mattias

Re: [hackers] [sbase][PATCH] grep: remove = flag from readme

2016-04-03 Thread FRIGN
On Wed, 30 Mar 2016 19:01:16 +0200 Mattias Andrée wrote: Hey Mattias, > $ echo äö | ./grep [å] > äö > > This is not want one expects from > a program that supports UTF-8. I assume the proper way here would be to add a note on the manpage that this highly depends on the

[hackers] [sbase][PATCH] grep: remove = flag from readme

2016-03-30 Thread Mattias Andrée
$ echo äö | ./grep [å] äö This is not want one expects from a program that supports UTF-8. Signed-off-by: Mattias Andrée --- README | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/README b/README index d60d8fc..bea2b0f 100644 --- a/README +++ b/README