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>
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

- Raw text -