net: skb_peek()/skb_peek_tail() cleanups
authorEric Dumazet <edumazet@google.com>
Mon, 30 Apr 2012 16:31:46 +0000 (16:31 +0000)
committerDavid S. Miller <davem@davemloft.net>
Tue, 1 May 2012 13:39:48 +0000 (09:39 -0400)
remove useless casts and rename variables for less confusion.

Signed-off-by: Eric Dumazet <edumazet@google.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
include/linux/skbuff.h

index 2c75e98953b28128bd796b433feb5d17d77605a3..988fc49667b13ef16f8d059811bc90af78b1d149 100644 (file)
@@ -883,10 +883,11 @@ static inline struct sk_buff *skb_unshare(struct sk_buff *skb,
  */
 static inline struct sk_buff *skb_peek(const struct sk_buff_head *list_)
 {
-       struct sk_buff *list = ((const struct sk_buff *)list_)->next;
-       if (list == (struct sk_buff *)list_)
-               list = NULL;
-       return list;
+       struct sk_buff *skb = list_->next;
+
+       if (skb == (struct sk_buff *)list_)
+               skb = NULL;
+       return skb;
 }
 
 /**
@@ -902,6 +903,7 @@ static inline struct sk_buff *skb_peek_next(struct sk_buff *skb,
                const struct sk_buff_head *list_)
 {
        struct sk_buff *next = skb->next;
+
        if (next == (struct sk_buff *)list_)
                next = NULL;
        return next;
@@ -922,10 +924,12 @@ static inline struct sk_buff *skb_peek_next(struct sk_buff *skb,
  */
 static inline struct sk_buff *skb_peek_tail(const struct sk_buff_head *list_)
 {
-       struct sk_buff *list = ((const struct sk_buff *)list_)->prev;
-       if (list == (struct sk_buff *)list_)
-               list = NULL;
-       return list;
+       struct sk_buff *skb = list_->prev;
+
+       if (skb == (struct sk_buff *)list_)
+               skb = NULL;
+       return skb;
+
 }
 
 /**