* [PATCH net-2.6.25] [NET]: Remove obsolete comment
@ 2008-01-05 12:21 Ilpo Järvinen
2008-01-06 7:18 ` David Miller
0 siblings, 1 reply; 2+ messages in thread
From: Ilpo Järvinen @ 2008-01-05 12:21 UTC (permalink / raw)
To: David Miller; +Cc: Netdev
[-- Attachment #1: Type: TEXT/PLAIN, Size: 1017 bytes --]
[PATCH] [NET]: Remove obsolete comment
It seems that ip_build_xmit is no longer used in here and
ip_append_data is used.
Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@helsinki.fi>
---
Hi Dave,
While reading some nearby code, I noticed this. I'm not 100%
sure if the removal is valid or not nor have interest to dig
it up from history but I suppose you know it better without
having to look it up from history :-).
net/ipv4/ip_output.c | 2 --
1 files changed, 0 insertions(+), 2 deletions(-)
diff --git a/net/ipv4/ip_output.c b/net/ipv4/ip_output.c
index 3dc0c12..e57de0f 100644
--- a/net/ipv4/ip_output.c
+++ b/net/ipv4/ip_output.c
@@ -1339,8 +1339,6 @@ static int ip_reply_glue_bits(void *dptr, char *to, int offset,
*
* Should run single threaded per socket because it uses the sock
* structure to pass arguments.
- *
- * LATER: switch from ip_build_xmit to ip_append_*
*/
void ip_send_reply(struct sock *sk, struct sk_buff *skb, struct ip_reply_arg *arg,
unsigned int len)
--
1.5.0.6
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH net-2.6.25] [NET]: Remove obsolete comment
2008-01-05 12:21 [PATCH net-2.6.25] [NET]: Remove obsolete comment Ilpo Järvinen
@ 2008-01-06 7:18 ` David Miller
0 siblings, 0 replies; 2+ messages in thread
From: David Miller @ 2008-01-06 7:18 UTC (permalink / raw)
To: ilpo.jarvinen; +Cc: netdev
From: "Ilpo_Järvinen" <ilpo.jarvinen@helsinki.fi>
Date: Sat, 5 Jan 2008 14:21:48 +0200 (EET)
> [PATCH] [NET]: Remove obsolete comment
>
> It seems that ip_build_xmit is no longer used in here and
> ip_append_data is used.
>
> Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@helsinki.fi>
> ---
>
> Hi Dave,
>
> While reading some nearby code, I noticed this. I'm not 100%
> sure if the removal is valid or not nor have interest to dig
> it up from history but I suppose you know it better without
> having to look it up from history :-).
It's definitely obsolete cruft, good catch, applied :-)
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2008-01-06 7:18 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-01-05 12:21 [PATCH net-2.6.25] [NET]: Remove obsolete comment Ilpo Järvinen
2008-01-06 7:18 ` David Miller
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox