From mboxrd@z Thu Jan  1 00:00:00 1970
Message-ID: <44CA0FE7.5020707@asgaard.homelinux.org>
Date: Fri, 28 Jul 2006 15:23:51 +0200
From: =?UTF-8?B?Ik5pbHMgTy4gU2Vsw6VzZGFsIg==?= <noselasd@asgaard.homelinux.org>
User-Agent: Thunderbird 1.5.0.4 (X11/20060614)
MIME-Version: 1.0
To: Fans of the OS Plan 9 from Bell Labs <9fans@cse.psu.edu>
Subject: Re: [9fans] quote file name
References: <939ccc98024339cb7c2c78e50894d9ec@quanstro.net>
	<92D74425-59CB-473D-90A9-CF49BCAC892D@ar.aichi-u.ac.jp>
In-Reply-To: <92D74425-59CB-473D-90A9-CF49BCAC892D@ar.aichi-u.ac.jp>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Topicbox-Message-UUID: 91d7fed6-ead1-11e9-9d60-3106f5b1d025

arisawa@ar.aichi-u.ac.jp wrote:
> Hello,
>
>> ; diff -c /n/sources/plan9/sys/src/libc/port/needsrcquote.c
>> needsrcquote.c
>> /n/sources/plan9/sys/src/libc/port/needsrcquote.c:6,12 -
>> needsrcquote.c:6,12
>>   {
>>       if(c <= ' ')
>>           return 1;
>> -     if(strchr("`^#*[]=|\\?${}()'<>&;", c))
>> +     if(c < 0x80 && strchr("`^#*[]=|\\?${}()'<>&;", c))
>>           return 1;
>>       return 0;
>>   }
>>
>> - erik
>
> Thanks eric. That works.
>
> I think kanji space (0x3000) should be quoted. Therefore
> -      if(c <= ' ')
> +       if(c <= ' ' || c == 0x3000)
> is desirable
>
> I hope official needsrcquote.c is to be changed along the patch by eric
> and me.

Surly the only space that needs concern is the "ascii"/U+0020 space, as
that's
used as a separator, not all the other unicode "space" characters. ?