On Fri, 6 Nov 2020 01:42:37 -0500 menglong8.d...@gmail.com wrote: > From: Menglong Dong <dong.mengl...@zte.com.cn> > > The initialization for 'err' with '-EINVAL' is redundant and > can be removed, as it is updated soon. > > Signed-off-by: Menglong Dong <dong.mengl...@zte.com.cn>
How many changes like this are there in the kernel right now? I'm afraid that if there are too many it's not worth the effort. Also - what tool do you use to find those, we need to make sure new instances don't get into the tree. > diff --git a/net/ipv4/devinet.c b/net/ipv4/devinet.c > index 123a6d3..847cb18 100644 > --- a/net/ipv4/devinet.c > +++ b/net/ipv4/devinet.c > @@ -651,7 +651,7 @@ static int inet_rtm_deladdr(struct sk_buff *skb, struct > nlmsghdr *nlh, > struct ifaddrmsg *ifm; > struct in_ifaddr *ifa; > You can remove this empty line while at it. > - int err = -EINVAL; > + int err; > > ASSERT_RTNL(); >
