On Thu, Mar 19, 2009 at 11:36:50AM +0100, Johannes Schindelin wrote: > > +void sort_string_list_with_fn(struct string_list *list, > > + int (*fn)(const void *, const void *)) > > +{ > > + qsort(list->items, list->nr, sizeof(*list->items), fn); > > +} > > + > > Do we really want an API for that? Calling qsort() directly should be > obvious enough, no? I'm fine with that, too. I was just encapsulating out of habit. But if it is normal to peek directly at list->nr and list->items, then calling qsort directly is fine. -Peff -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html