[PATCH] move inline keywords nearer beginning of declaration inskbuff.c to elliminate warning with gcc -W

From: Jesper Juhl
Date: Tue Nov 23 2004 - 19:57:22 EST



In order to get rid of
net/core/skbuff.c:1353: warning: `inline' is not at beginning of declaration
net/core/skbuff.c:1374: warning: `inline' is not at beginning of declaration
when building with gcc -W, I submit the patch below.
There's no impact on the way the code works, so it is perfectly safe. It
just brings the count of warnings that I have to sift through down by two
(yes, I know the kernel is not normally build with -W, but I do it to
look for stuff that potentially needs review, and the less warnings I
have to sift through the better). I see no good reason not to apply this.
Please apply.

Signed-off-by: Jesper Juhl <juhl-lkml@xxxxxx>

diff -up linux-2.6.10-rc2-bk6-orig/net/core/skbuff.c linux-2.6.10-rc2-bk6/net/core/skbuff.c
--- linux-2.6.10-rc2-bk6-orig/net/core/skbuff.c 2004-11-17 01:20:32.000000000 +0100
+++ linux-2.6.10-rc2-bk6/net/core/skbuff.c 2004-11-24 01:05:32.000000000 +0100
@@ -1350,7 +1350,7 @@ void skb_add_mtu(int mtu)
}
#endif

-static void inline skb_split_inside_header(struct sk_buff *skb,
+static inline void skb_split_inside_header(struct sk_buff *skb,
struct sk_buff* skb1,
const u32 len, const int pos)
{
@@ -1371,7 +1371,7 @@ static void inline skb_split_inside_head
skb->tail = skb->data + len;
}

-static void inline skb_split_no_header(struct sk_buff *skb,
+static inline void skb_split_no_header(struct sk_buff *skb,
struct sk_buff* skb1,
const u32 len, int pos)
{



-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/