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

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.