Thread (14 messages) 14 messages, 6 authors, 2014-03-29

Re: [PATCH net-next v2 3/3] ipv6: fix checkpatch errors of "foo*" and "foo * bar"

From: David Miller <davem@davemloft.net>
Date: 2014-03-28 17:09:41

From: wangyufen <redacted>
Date: Fri, 28 Mar 2014 17:33:54 +0800
On 2014/3/28 13:47, Joe Perches wrote:
quoted
On Fri, 2014-03-28 at 01:40 -0400, David Miller wrote:
quoted
From: Joe Perches <joe@perches.com>
Date: Thu, 27 Mar 2014 21:22:01 -0700
quoted
On Fri, 2014-03-28 at 12:07 +0800, Wangyufen wrote:
[]
quoted
diff --git a/net/ipv6/ip6_fib.c b/net/ipv6/ip6_fib.c
[]
quoted
@@ -143,7 +143,7 @@ static __inline__ __be32 addr_bit_set(const void *token, int fn_bit)
 	       addr[fn_bit >> 5];
 }
Perhaps all the __inline__ uses could be changed to inline too.
Or rather, deleted completely, this is a *.c file after all.
Maybe right.  There are a lot though just in net/

$ git ls-files net | grep "\.c$" | \
  xargs grep -Pw "_{0,2}inline_{0,2}" | wc -l
1512



Or change to inline and move the whole function to ip6_fib.h?
No, that is not desirable.  The whole point is to not export the interface
outside of the *.c file, and to let the compiler make inlining decisions
since it can see all of the call sites.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help