In article <[EMAIL PROTECTED]> (at Mon, 05 Feb 2007 17:32:58 -0800 (PST)), David Miller <[EMAIL PROTECTED]> says:
> From: YOSHIFUJI Hideaki <[EMAIL PROTECTED]> > Date: Tue, 06 Feb 2007 10:24:05 +0900 (JST) > > > > @@ -498,7 +500,8 @@ static void ndisc_send_na(struct net_device *dev, > > > struct neighbour *neigh, > > > msg->icmph.icmp6_unused = 0; > > > msg->icmph.icmp6_router = router; > > > msg->icmph.icmp6_solicited = solicited; > > > - msg->icmph.icmp6_override = override; > > > + msg->icmph.icmp6_override = override; > > > + > > > > > > /* Set the target address. */ > > > ipv6_addr_copy(&msg->target, solicited_addr); > > > > Why do you "change" this? > > He edited it, but then reverted his change but in the > process added proper tab characters :-) It is easy > to miss things like this. I know, you know. :-) > This whole file has a lot of lines lacking proper tab > characters. Yes, I agree, but I think it is different issue, and maybe, we should check and change other places as well, in separate patch(es). --yoshfuji - 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