Re: ALSA: ctxfi: Delete an unnecessary check before kfree()

Takashi Iwai
2014-12-03T17:30:08Z

At Wed, 03 Dec 2014 09:14:48 -0800,
Joe Perches wrote:
>
> On Wed, 2014-12-03 at 13:41 +0100, Takashi Iwai wrote:
> > At Wed, 03 Dec 2014 12:38:51 +0100,
> > SF Markus Elfring wrote:
> > >
> > > >> Deletion of an unnecessary check before the function call "kfree"
> > > >> One function call less in get_daio_rsc() after error detection
> > > >
> > > > In these cases, the changes aren't so straightforward, and they don't
> > > > improve the readability.
> > >
> > > How do you prefer to improve the affected source code here?
> >
> > Don't touch then. There are no bugs to fix there.
>
> Takashi, what did you think of this?
> https://lkml.org/lkml/2014/12/2/771
>
> Just unnecessary?

Well, this one looks more consistent. But honestly speaking, it's
rather a matter of taste. So I'm not so much inclined to merge the
stuff, too, sorry. If it's proven to reduce the compiled size, etc,
I'll happily apply it, though.

FWIW, what wasn't good in the original patch was to break the balance.
It removed only the check for dai, and not for dao. One would wonder
why there is a check only for one.

It could be two simple kfree() calls instead. But then this won't be
an improvement, as it gets one more function call, which is more
expensive than a conditional.

Takashi