On Mon, Jul 11, 2022 at 2:40 PM Nuno Sá <nuno.sa@xxxxxxxxxx> wrote: > > Just cosmetics. No functional change intended... ... > - if (name == NULL && channel_name == NULL) > + if (!name && !channel_name) > return ERR_PTR(-ENODEV); After this change in place, I think it's better to convert it to if (!(name || channel_name)) which shows intentions clearer. -- With Best Regards, Andy Shevchenko