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