Re: tree(7) man page diff

2010-05-05 Thread Nicholas Marriott
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

Re: tree(7) man page diff

2010-05-04 Thread Tim van der Molen
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".

Re: tree(7) man page diff

2010-05-03 Thread Nicholas Marriott
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

tree(7) man page diff

2010-05-03 Thread Tim van der Molen
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 =