Hi On Thu, Oct 8, 2015 at 1:31 PM, Sergei Zviagintsev <[email protected]> wrote: > Use list_next_entry() instead of list_first_entry().
This commit-message is completely useless. This is not a cosmetic change, so please justify your changes. Sure, in this case both list_next_entry() and list_first_entry() perform the same operation, but you should really describe such changes in the commit message to allow reviewers to understand the intent of your change. Anyway, patch looks good, thanks! David > Signed-off-by: Sergei Zviagintsev <[email protected]> > --- > ipc/kdbus/queue.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/ipc/kdbus/queue.c b/ipc/kdbus/queue.c > index f9c44d7bae6d..90e8d16f5967 100644 > --- a/ipc/kdbus/queue.c > +++ b/ipc/kdbus/queue.c > @@ -157,8 +157,7 @@ static void kdbus_queue_entry_unlink(struct > kdbus_queue_entry *entry) > * the list. Update cached highest-priority entry, store the > * new one as the tree node. > */ > - q = list_first_entry(&entry->prio_entry, > - struct kdbus_queue_entry, prio_entry); > + q = list_next_entry(entry, prio_entry); > list_del(&entry->prio_entry); > > if (queue->msg_prio_highest == &entry->prio_node) > -- > 1.8.3.1 > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
