Hi Vinicius, On Fri, Jul 27, 2012 at 6:52 PM, Anderson Lizardo <anderson.lizardo@xxxxxxxxxxxxx> wrote: > Does it make sense to merge the two if()'s above into one? i.e.: > > if (IS_ERR(acl) || type == ACL_LINK) > return acl; > > It's still clear, IMHO. Nevermind, now I see you just factored out this code on next patch. Regards, -- Anderson Lizardo Instituto Nokia de Tecnologia - INdT Manaus - Brazil -- 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