On Mon, Dec 12, 2011 at 11:55:57PM +0100, Davidlohr Bueso wrote: > One note: the code is added to the header file instead of its own C file > so we don't have to update all the build rules of the list.h users. [...] > > +static struct list_head *merge(int (*cmp)(struct list_head *a, > > + struct list_head *b), > > + struct list_head *a, struct list_head *b) shouldn't be the function "static inline" (or _INLINE_ ) ? -- Karel Zak <kzak@xxxxxxxxxx> http://karelzak.blogspot.com -- To unsubscribe from this list: send the line "unsubscribe util-linux" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html