Re: CFT: BSD-licensed grep [Fwd: cvs commit: ports/textproc/bsdgrep Makefile distinfo]

From: Andrey Chernov <ache_at_nagual.pp.ru>
Date: Sun, 22 Jun 2008 17:53:43 +0400
On Sun, Jun 22, 2008 at 02:58:17PM +0200, Gabor Kovesdan wrote:
> Andrey Chernov escribi?:
> > On Wed, Jun 18, 2008 at 12:40:24PM +0200, Dag-Erling Sm??rgrav wrote:
> >   
> >> For grep, I believe it should simply be a matter of calling setlocale(),
> >> using wide strings, and using a multibyte regex engine (for appropriate
> >> values of "simply").
> >>     
> >
> > See my prev reply telling more details. Using wide strings is not so easy, 
> > f.e. all ctype BSD grep now uses should be converted to wctype, input 
> > conversion added, etc.
> >   
> I've started to work on doing this big change, the first step:
> http://kovesdan.org/patches/grep-i18n.diff

1) You can't convert just whole buffer after fread() since it can be 
ended in the middle of multibyte sequence on BUFSIZ edge. Look how GNU 
utils do it.

2) Better use iswspace and iswcntrl instead of iswctype.

3) util.c needs to be fixed in several places too.

-- 
http://ache.pp.ru/
Received on Sun Jun 22 2008 - 11:53:55 UTC

This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:39:32 UTC