2008-08-12 06:14:04

by Jean-Christophe Dubois

[permalink] [raw]
Subject: [PATCH] remove unnecessary variable in xfrm_output_resume()

Small fix removing an unnecessary intermediate variable.

Signed-off-by: Jean-Christophe DUBOIS <[email protected]>
---

diff -Naur linux-2.6.26.2.org/net/xfrm/xfrm_output.c linux-2.6.26.2/net/xfrm/xfrm_output.c
--- linux-2.6.26.2.org/net/xfrm/xfrm_output.c 2008-08-12 00:37:29.000000000 +0200
+++ linux-2.6.26.2/net/xfrm/xfrm_output.c 2008-08-12 00:40:40.000000000 +0200
@@ -112,16 +112,13 @@
int xfrm_output_resume(struct sk_buff *skb, int err)
{
while (likely((err = xfrm_output_one(skb, err)) == 0)) {
- struct xfrm_state *x;


2008-08-13 01:16:07

by Herbert Xu

[permalink] [raw]
Subject: Re: [PATCH] remove unnecessary variable in xfrm_output_resume()

Jean-Christophe Dubois <[email protected]> wrote:
> Small fix removing an unnecessary intermediate variable.
>
> Signed-off-by: Jean-Christophe DUBOIS <[email protected]>

Please send this to [email protected] and cc [email protected].

Thanks,
--
Visit Openswan at http://www.openswan.org/
Email: Herbert Xu ~{PmV>HI~} <[email protected]>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt

2008-08-13 05:50:20

by Jean-Christophe Dubois

[permalink] [raw]
Subject: Re: [PATCH] remove unnecessary variable in xfrm_output_resume()

Le Wednesday 13 August 2008 03:16:04 Herbert Xu, vous avez ?crit?:
> Jean-Christophe Dubois <[email protected]> wrote:
> > Small fix removing an unnecessary intermediate variable.
> >
> > Signed-off-by: Jean-Christophe DUBOIS <[email protected]>
>
> Please send this to [email protected] and cc [email protected].

Done

Thanks

>
> Thanks,

2008-08-13 05:53:42

by David Miller

[permalink] [raw]
Subject: Re: [PATCH] remove unnecessary variable in xfrm_output_resume()

From: Jean-Christophe Dubois <[email protected]>
Date: Wed, 13 Aug 2008 07:50:16 +0200

> Le Wednesday 13 August 2008 03:16:04 Herbert Xu, vous avez ?crit?:
> > Jean-Christophe Dubois <[email protected]> wrote:
> > > Small fix removing an unnecessary intermediate variable.
> > >
> > > Signed-off-by: Jean-Christophe DUBOIS <[email protected]>
> >
> > Please send this to [email protected] and cc [email protected].
>
> Done

Ugh, I didn't see Herbert tell you this sorry :-)

I do really already have a copy of this in my backlog.

2008-08-13 09:38:13

by David Miller

[permalink] [raw]
Subject: Re: [PATCH] remove unnecessary variable in xfrm_output_resume()

From: Jean-Christophe Dubois <[email protected]>
Date: Tue, 12 Aug 2008 08:12:24 +0200

> Small fix removing an unnecessary intermediate variable.
>
> Signed-off-by: Jean-Christophe DUBOIS <[email protected]>

Your patch is whitespace mangled by your email client, the tab
characters have been corrupted into spaces, therefore your patch
will not apply.

Please fix this up and resubmit.

Thanks.

2008-08-13 18:50:21

by Jean-Christophe Dubois

[permalink] [raw]
Subject: Re: [PATCH] remove unnecessary variable in xfrm_output_resume()

Le Wednesday 13 August 2008 11:38:14 David Miller, vous avez ?crit?:
> From: Jean-Christophe Dubois <[email protected]>
> Date: Tue, 12 Aug 2008 08:12:24 +0200
>
> > Small fix removing an unnecessary intermediate variable.
> >
> > Signed-off-by: Jean-Christophe DUBOIS <[email protected]>
>
> Your patch is whitespace mangled by your email client, the tab
> characters have been corrupted into spaces, therefore your patch
> will not apply.
>
> Please fix this up and resubmit.

Sorry, I'll resubmit.

JC

>
> Thanks.
> --
> To unsubscribe from this list: send the line "unsubscribe netdev" in
> the body of a message to [email protected]
> More majordomo info at http://vger.kernel.org/majordomo-info.html