Hi Mikel, On Tue, Mar 19, 2013, Mikel Astiz wrote: > +static gint service_prio_cmp(gconstpointer a, gconstpointer b) > { > - const struct btd_profile *p1 = a, *p2 = b; > + struct btd_service *s1 = (gpointer) a; > + struct btd_service *s2 = (gpointer) b; Again, adding const specifiers here would help avoid the explicit casting. Johan -- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html