[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Coverage for libltdl/slist.c and fallout
From: |
Gary V. Vaughan |
Subject: |
Re: Coverage for libltdl/slist.c and fallout |
Date: |
Mon, 30 Nov 2009 13:55:48 -0500 |
Typo alert:
On 30 Nov 2009, at 12:29, Gary V. Vaughan wrote:
> if you are proposing that SListCallback functions passed to
> slist_remove should always return SList *, then that does seem like a
> worthy addition to me. In that case, the following is cleaner, and
> goes some way towards preventing misuse of the API by having the find
> callback return something from the innards of an element (which is,
> by the way still a valid use-case for other SListCallback occurences):
>
> libltd/libltdl/slist.h:
> typedef SList * SListRemoveCallback (SList *item, void *userdata);
> ...
> LT_SCOPE void * slist_remove (SList **phead, SListRemoveCallback *find,
> void *matchdata);
Of course that should have said:
LT_SCOPE SList * slist_remove...
> libltdl/slist.c:
> SList *
> slist_remove (SList **phead, SListRemoveCallback *find, void *matchdata)
> {
> SList *stale = 0;
> SList *result = 0;
>
> ...rest of slist_remove is untouched...
> }
Cheers,
Gary
--
Gary V. Vaughan (address@hidden)