Mailing-List: contact cygwin-developers-help AT sourceware DOT cygnus DOT com; run by ezmlm
List-Subscribe: <mailto:cygwin-developers-subscribe AT sources DOT redhat DOT com>
List-Archive: <http://sources.redhat.com/ml/cygwin-developers/>
List-Post: <mailto:cygwin-developers AT sources DOT redhat DOT com>
List-Help: <mailto:cygwin-developers-help AT sources DOT redhat DOT com>, <http://sources.redhat.com/ml/#faqs>
Sender: cygwin-developers-owner AT sources DOT redhat DOT com
Delivered-To: mailing list cygwin-developers AT sources DOT redhat DOT com
Date: Thu, 31 May 2001 14:03:13 -0400
From: Christopher Faylor <cgf AT redhat DOT com>
To: cygwin-developers AT cygwin DOT com
Subject: Re: isspace() & i18n
Message-ID: <20010531140313.C23914@redhat.com>
Reply-To: cygwin-developers AT cygwin DOT com
Mail-Followup-To: cygwin-developers AT cygwin DOT com
References: <13254193676 DOT 20010530145756 AT logos-m DOT ru> <20010530095254 DOT B17603 AT redhat DOT com> <9265832872 DOT 20010530181156 AT logos-m DOT ru> <20010530101516 DOT E17603 AT redhat DOT com> <3066562491 DOT 20010530182405 AT logos-m DOT ru> <3B151688 DOT B6ABA6FD AT yahoo DOT com>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
User-Agent: Mutt/1.3.11i
In-Reply-To: <3B151688.B6ABA6FD@yahoo.com>; from earnie_boyd@yahoo.com on Wed, May 30, 2001 at 11:49:28AM -0400

This has been checked in.  It doesn't solve the larger issue of
if this is a newlib bug or not but I think that this is a correct
fix nonetheless.

Thanks, Earnie.

cgf

On Wed, May 30, 2001 at 11:49:28AM -0400, Earnie Boyd wrote:
>So does this solve the problem?
>
>-- 
>Earnie.
>Index: path.cc
>===================================================================
>RCS file: /cvs/src/src/winsup/cygwin/path.cc,v
>retrieving revision 1.139
>diff -u -p -r1.139 path.cc
>--- path.cc	2001/05/14 02:52:12	1.139
>+++ path.cc	2001/05/30 15:45:49
>@@ -2929,7 +2929,7 @@ chdir (const char *dir)
>      whitespace to SetCurrentDirectory.  This doesn't work too well
>      with other parts of the API, though, apparently.  So nuke trailing
>      white space. */
>-  for (s = strchr (dir, '\0'); --s >= dir && isspace (*s); )
>+  for (s = strchr (dir, '\0'); --s >= dir && isspace ((unsigned int)*s); )
>     *s = '\0';
> 
>   if (path.error)