Message-ID: <39479072.91244E32@softhome.net>
Date: Wed, 14 Jun 2000 17:02:26 +0300
From: Laurynas Biveinis <lauras AT softhome DOT net>
X-Mailer: Mozilla 4.73 [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 <djgpp-workers AT delorie DOT com>
Subject: Re: Another one symlink patch
References: <Pine DOT SUN DOT 3 DOT 91 DOT 1000614173902 DOT 995J-100000 AT is>
Content-Type: text/plain; charset=us-ascii
Content-Transfer-Encoding: 7bit
Reply-To: djgpp-workers AT delorie DOT com

Eli Zaretskii wrote:
> > As I understand texinfo, @item specifies just a label in a table layout
> > and the real EWHATEVER value does not matter there, does it?
> 
> No.  This is what @item in a @table produces:
[...]
> So it shows both the numerical value and the text.

OK then.

> > I was 110% sure that I've included missing S_ISLNK *this time*.
> > Guess if it is really so. But the real code sets all bits correctly, it
> > looks like this:

> I don't see this code inside `lstat' in your patches.  In fact, I don't
> see a call to `readlink' inside `lstat' at all (and I didn't expect to).

I wanted to say, it has slipped from the patch *again*. And readlink(...)
was used not to read link but to check if the file is symlink. I will
replace that with __is_symlink().

> I agree, but this probably should be mentioned in the docs as another way
> of checking `readlink's results.

OK.

Laurynas