Date: Thu, 14 Jun 2001 22:18:04 +0300
From: "Eli Zaretskii" <eliz AT is DOT elta DOT co DOT il>
Sender: halo1 AT zahav DOT net DOT il
To: lauras AT softhome DOT net
Message-Id: <7458-Thu14Jun2001221803+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: djgpp-workers AT delorie DOT com
In-reply-to: <20010524195242.A236@lauras.lt>
Subject: Re: Add d_type member to struct dirent
References: <20010523210505 DOT A478 AT lauras DOT lt> <Pine DOT SUN DOT 3 DOT 91 DOT 1010524084349 DOT 17472E-100000 AT is> <20010524195242 DOT A236 AT lauras DOT lt>
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: "Laurynas Biveinis" <lauras AT softhome DOT net>
> Date: Thu, 24 May 2001 19:52:42 +0200
> 
> > > ...but in 99.9% cases __internal_readlink() will call just filelength()
> > > to check for 510 bytes magic size. No open() at all in this case, and plain
> > > filelength() is cheaper that stat(), isn't it?
> > 
> > ??? I don't have the latest CVS sources handy where I type this, but IIRC 
> > filelength takes a handle, so __internal_readlink needs to open the file, 
> > before it knows that it's 510 byte long.
> 
> What I forgot to tell is that __internal_readlink does findfirst() to
> find out file size if it has path instead of the handle. Does my point
> still hold true then?

I ended up opening the file and passing a handle to
__internal_readlink.  It seemed simpler.

Thanks for the feedback.