Source-Changes archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: CVS commit: src/lib/libc/gen



On Sat, Dec 31, 2005 at 02:17:00AM +0200, Elad Efrat wrote:
> Quentin Garnier wrote:
> 
> > Please re-re-re-revert (or whatever) that change.
> 
> Test attached patch.

Well, that fixes potential issues from leaving errno in a weird value,
but that doesn't fix the fact that we don't return NULL and set errno to
ENOENT when a path component doesn't exist.

That was the very subject of lib/30943.  In the end you agreed on the
behaviour realpath(3) should have, and suddenly you're changing your
mind because it's convenient for you?  How reasonable is that?

At least, re-open the PR when you have such a mind change.  Inverting
the behaviour of a function does not help in keeping bugs away IMO.
Quite the contrary.

-- 
Quentin Garnier - cube%cubidou.net@localhost - cube%NetBSD.org@localhost
"When I find the controls, I'll go where I like, I'll know where I want
to be, but maybe for now I'll stay right here on a silent sea."
KT Tunstall, Silent Sea, Eye to the Telescope, 2004.

Attachment: pgpXn6etBT6Gw.pgp
Description: PGP signature



Home | Main Index | Thread Index | Old Index