[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]
Re: lib/39684 (RB_REMOVE trashes link to tree, when requested to remove object that isn't in tree)
Documenting features *correctly* is okay, returning NULL on error is
excellent and not trashing tree is superb.
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.
Life is complex. It has real and imaginary parts.
Main Index |
Thread Index |