Messages in this thread | | | Date | Fri, 3 Apr 2020 10:25:21 +0100 | From | Lee Jones <> | Subject | Re: [PATCH] mfd: asic3: Delete redundant variable definition |
| |
On Fri, 03 Apr 2020, Markus Elfring wrote:
> > In this function, 'ret' is always assigned, even if 'pdata->leds' > > don't carry out, > > I notice possibilities again to improve such a commit message. > > > > it has already been assigned a value in the above code, including '0', > > The variable assignment will eventually be performed only in if branches. > > > > so it's redundant. > > I suggest to reconsider this interpretation of the source code here. > Would you like to move the mentioned statement into an else branch > at the end?
Why wasn't this sent 'in reply to'? Something wrong with your mailer?
Now this messages is orphaned. Floating about in my inbox on its own.
-- Lee Jones [李琼斯] Linaro Services Technical Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog
| |