Message-ID:	<39A2CBFF.5598DBA4@softhome.net>
Date:	Tue, 22 Aug 2000 20:52:47 +0200
From:	Laurynas Biveinis <lauras AT softhome DOT net>
X-Mailer:	Mozilla 4.74 [en] (Win98; U)
X-Accept-Language:	lt,en
MIME-Version:	1.0
To:	Eli Zaretskii <eliz AT is DOT elta DOT co DOT il>
CC:	djgpp-workers AT delorie DOT com
Subject:	Re: Patch: symlinks in _dxe_load()
References:	<39A275AD DOT 31F59BA0 AT softhome DOT net> <3995-Tue22Aug2000202653+0300-eliz AT is DOT elta DOT co DOT il>
Reply-To:	djgpp-workers AT delorie DOT com

Eli Zaretskii wrote:
> Hmm.. I'm not sure this is a good idea.  _dxe_load was meant to be as
> small as possible.  Why does it need to support symlinks?

Consistency. But if this is too high price, I don't mind doing nothing
there.

> (And, btw: if you replace _open with open, you need to use O_BINARY,
> since that's how _open opens the file.  

OTOH, if we would add symlink support here, it would be better to leave
_open() and add __solve_symlinks(). The overhead should be smaller than
with open() in this case.

Laurynas

- Raw text -