On Tue, May 04, 2010 at 09:25:29PM +0200, Tim van der Molen wrote:
> On Tue, 04 May 2010 00:20:47 +0200, Nicholas Marriott wrote:
> > SPLAY_REMOVE appears to returns the pointer it is given or NULL if the
> > item isn't found. I don't see what errors it could return NULL for, so I
> > think it woul
On Tue, 04 May 2010 00:20:47 +0200, Nicholas Marriott wrote:
> SPLAY_REMOVE appears to returns the pointer it is given or NULL if the
> item isn't found. I don't see what errors it could return NULL for, so I
> think it would be better to say "NULL is returned if no item with that
> key is found".
I like this idea, and the parts for SPLAY_INSERT and RB_INSERT look
fine.
SPLAY_REMOVE appears to returns the pointer it is given or NULL if the
item isn't found. I don't see what errors it could return NULL for, so I
think it would be better to say "NULL is returned if no item with that
key is fo
This diff moves the documentation of the return values of
{RB,SPLAY}_{INSERT,REMOVE} from the NOTES section (where it is hard to
find) to the place where each function is described. (Note that the
RB_INSERT description already documents the return values.)
Regards,
Tim
Index: tree.3
=