On Thu, 24 Sep 2009, Rusty Russell wrote: > On Thu, 24 Sep 2009 02:58:45 am Tim Abbott wrote: > > +void *bsearch(const void *key, const void *base, size_t num, size_t size, > > + int (*cmp)(const void *key, const void *elt)) > > +{ > > + int start = 0, end = num - 1, mid, result; > > + if (num == 0) > > + return NULL; > > + > > + while (start <= end) { > > The if (num == 0) line is superfluous. You are quite right. -Tim Abbott -- To unsubscribe from this list: send the line "unsubscribe linux-modules" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html