Subject: Re: ls(1) support for multibyte chars
To: SODA Noriyuki <soda@sra.co.jp>
From: Thomas Klausner <wiz@NetBSD.org>
List: tech-userlevel
Date: 06/05/2007 19:48:30
On Thu, May 31, 2007 at 03:39:13AM +0900, SODA Noriyuki wrote:
> >>>>> On Tue, 29 May 2007 18:38:22 +0200,
>       Thomas Klausner <wiz@NetBSD.org> said:
> 
> > In a private discussion, SODA Noriyuki provided the attached patch for
> > adding multibyte support to ls(1).
> > 
> > Is it ok to commit?
> > 
> > +		if (span > MB_CUR_MAX)
> > +			span = MB_CUR_MAX;
> 
> It seems to be better to make the above part commented out by "#if 0".
> Because if there are reduandant escape sequences which exceeds 32 bytes,
> our current implementation doesn't display the pathname correctly with
> above.

Ok, I committed it with that additional change.

Thanks you for the patch!
 Thomas