Mailing-List:	contact cygwin-apps-help AT cygwin DOT com; run by ezmlm
Sender:	cygwin-apps-owner AT cygwin DOT com
List-Subscribe:	<mailto:cygwin-apps-subscribe AT cygwin DOT com>
List-Archive:	<http://sources.redhat.com/ml/cygwin-apps/>
List-Post:	<mailto:cygwin-apps AT cygwin DOT com>
List-Help:	<mailto:cygwin-apps-help AT cygwin DOT com>, <http://sources.redhat.com/lists.html#faqs>
Mail-Followup-To:	cygwin-apps AT cygwin DOT com
Delivered-To:	mailing list cygwin-apps AT cygwin DOT com
Message-ID:	<3CEC3DB1.8010100@ece.gatech.edu>
Date:	Wed, 22 May 2002 20:54:09 -0400
From:	Charles Wilson <cwilson AT ece DOT gatech DOT edu>
User-Agent:	Mozilla/5.0 (X11; U; SunOS sun4u; en-US; rv:0.9.4.1) Gecko/20020406 Netscape6/6.2.2
X-Accept-Language:	en-us
MIME-Version:	1.0
To:	Matthew Smith <matts AT bluesguitar DOT org>
CC:	"Peter A. Castro" <doctor AT fruitbat DOT org>, cygwin-apps AT cygwin DOT com
Subject:	Re: new package offering: zsh
References:	<Pine DOT LNX DOT 4 DOT 21 DOT 0205221654580 DOT 12767-100000 AT gremlin DOT fruitbat DOT org> <021801c201f2$8e241d00$010a10ac AT bluesguitar DOT org>
X-Virus-Scanned:	by amavisd-milter (http://amavis.org/)

No -- if zsh can handle cr+lf internally, then let it do so.  Explicitly 
open function files in binary mode -- this will override the "textmode 
mount" which sets the default read mode.

That way, number of bytes reported by stat() will equal the number of 
bytes reported by read().  The buffer thus filled will contain cr+lf -- 
but supposedly zsh can deal with that...

--Chuck


Matthew Smith wrote:

> This sounds perfectly reasonable to me.
> 
> cheers,
> -Matt
> 
> 
>>I know of one issue concerning loading of functions.  Is there another
>>you had in mind?  For the most part, zsh handles cr+lf as if it were lf
>>implicitly.  For loading functions, the code assumes a binary mode, ie:
>>the number of bytes reported by stat() of a file is the number of bytes
>>reported back from read() after reading in the whole file.  On a
>>filesystem mounted in textmode, this isn't always true and can cause the
>>function to not load.  I really feel this is a filesystem attribute
>>issue, but if it's annoying enough for people, I'll loosen the size
>>matching restrictions.  A workaround for now is to strip your functions
>>of all Carriage-Returns, thus only have linefeeds in the file. 
>>
>>
> 
> 
> 

- Raw text -