Re: [PATCH net-next-2.6 4/7] sctp: remove useless arguments from get_saddr() call

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]


From: Wei Yongjun <yjwei@xxxxxxxxxxxxxx>
Date: Tue, 26 Apr 2011 11:47:39 +0800

> @@ -392,11 +392,11 @@ static inline int sctp_v6_addr_match_len(union sctp_addr *s1,
>   */
>  static void sctp_v6_get_saddr(struct sctp_sock *sk,
>  			      struct sctp_transport *t,
> -			      union sctp_addr *daddr,
>  			      struct flowi *fl)
>  {
>  	struct flowi6 *fl6 = &fl->u.ip6;
>  	union sctp_addr *saddr = &t->saddr;
> +	union sctp_addr *daddr = &t->ipaddr;
>  
>  	SCTP_DEBUG_PRINTK("%s: asoc:%p dst:%p daddr:%pI6 ",
>  			  __func__, t->asoc, t->dst, &daddr->v6.sin6_addr);

I really get grumpy when I have to fix up stuff like this:

net/sctp/ipv6.c: In function ʽsctp_v6_get_saddrʼ:
net/sctp/ipv6.c:382: warning: unused variable ʽdaddrʼ

You guys know I'm going to immediately run make on any patch you send
me and look for new warnings.

Why waste my time and not look for them yourselves before posting the
patch?

This wasn't even one of those cases where the warning goes away at
the end of the patch series, and only exists somewhere in the middle.

?τθΊ{.nΗ+?·???­?+%?Λ?±ιέΆ??w?Ί{.nΗ+?·??{±ώΗ-§ψ§Ά?ʽά¨}©?²Ζ zΪ&j:+v?¨ώψ―ω?w?ώ?ΰ2?ή?¨θ­Ϊ&ʼ)ίʽ«aΆΪ??ϋΰzΏδzΉή?ϊ+?ω???έʼj??wθώf



[Linux OMAP]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Photo]     [Yosemite News]    [Yosemite Photos]    [Free Online Dating]     [Linux Kernel]     [Linux SCSI]     [XFree86]

Add to Google Reader or Homepage Powered by Linux