Date: Sat, 29 Sep 2001 12:05:28 +0200
From: "Eli Zaretskii" <eliz AT is DOT elta DOT co DOT il>
Sender: halo1 AT zahav DOT net DOT il
To: sandmann AT clio DOT rice DOT edu
Message-Id: <7704-Sat29Sep2001120527+0300-eliz@is.elta.co.il>
X-Mailer: Emacs 20.6 (via feedmail 8.3.emacs20_6 I) and Blat ver 1.8.9
CC: wojciech DOT galazka AT polkomtel DOT com DOT pl, djgpp-workers AT delorie DOT com
In-reply-to: <10109290242.AA12401@clio.rice.edu> (sandmann@clio.rice.edu)
Subject: Re: fixpath patch (rev 3)
References:  <10109290242 DOT AA12401 AT clio DOT rice DOT edu>
Reply-To: djgpp-workers AT delorie DOT com
Errors-To: nobody AT delorie DOT com
X-Mailing-List: djgpp-workers AT delorie DOT com
X-Unsubscribes-To: listserv AT delorie DOT com
Precedence: bulk

> From: sandmann AT clio DOT rice DOT edu (Charles Sandmann)
> Date: Fri, 28 Sep 2001 21:42:36 -0500 (CDT)
> 
> New version in ftp://clio.rice.edu/djgpp/fixpathtest.zip
> (not quite final, needs clean up, review, copyrights, os_truever, etc).

Looks fine, but I thought we agreed that the carry flag should be set
before calling 4700h/7147h?

> It will only behave differently than the well tested fixpath if getcwd
> fails (which we didn't handle before - it now becomes relative path) or
> if Win2000 type OS (lfn and 0x532) and a seemingly root directory.

There's a subtle issue with _fixpath returning "d:." in these cases:
if a program chdirs between opendir and readdir, the wrong directory
will be read.  If we cannot fix that, we should at least document it.