Re: lib/39684 (RB_REMOVE trashes link to tree, when requested to remove object that isn't in tree)

["Matti Lehtonen" <m-Matti-a.Lehtonen%IKI.Fi@localhost>]

The following reply was made to PR lib/39684; it has been noted by GNATS.

From: "Matti Lehtonen" <m-Matti-a.Lehtonen%IKI.Fi@localhost>
To: gnats-bugs%netbsd.org@localhost
Cc: lib-bug-people%netbsd.org@localhost, gnats-admin%netbsd.org@localhost, 
netbsd-bugs%netbsd.org@localhost
Subject: Re: lib/39684 (RB_REMOVE trashes link to tree, when requested to 
remove object that isn't in tree)
Date: Wed, 8 Oct 2008 10:31:01 +0300

 Hi!
 
 Documenting features *correctly* is okay, returning NULL on error is
 excellent and not trashing tree is superb.
 
 Lehtonen, Matti
 
 2008/10/7 Christos Zoulas <christos%zoulas.com@localhost>:
 > The following reply was made to PR lib/39684; it has been noted by GNATS.
 >
 > From: christos%zoulas.com@localhost (Christos Zoulas)
 > To: Quentin Garnier <cube%cubidou.net@localhost>
 > Cc: gnats-bugs%NetBSD.org@localhost
 > Subject: Re: lib/39684 (RB_REMOVE trashes link to tree, when requested to 
 > remove object that isn't in tree)
 > Date: Tue, 7 Oct 2008 14:42:35 -0400
 >
 >  On Oct 7,  8:33pm, cube%cubidou.net@localhost (Quentin Garnier) wrote:
 >  -- Subject: Re: lib/39684 (RB_REMOVE trashes link to tree, when requested to
 >
 >  | I don't see how the submitter's use of the function is different from,
 >  | say, doing strlen(NULL).  And would you really think there is a need to
 >  | mention that you shouldn't do strlen(NULL) in strlen(3)?
 >  |
 >  | If so you have a huge documentation task ahead of you.
 >
 >  I am not planning to do this of course and I understand what you are saying.
 >  The issue here is that the documentation used to say that it would return
 >  NULL on error and the code did not. Now it has been fixed.
 >
 >  christos
 >
 >
 
 
 
 -- 
 Life is complex. It has real and imaginary parts.