Re: [PATCH 3/5] ISDN-Gigaset: Delete an error message for a failed memory allocation

From
Paul Bolle <hidden>
Date
2016-09-28 11:42:48
Also in
kernel-janitors, lkml
Reply to
Tilman Schmidt, 2016-09-27
On Tue, 2016-09-27 at 12:57 +0200, Tilman Schmidt wrote:
On Mon, Sep 26, 2016, at 17:42, SF Markus Elfring wrote:
quoted
Omit an extra message for a memory allocation failure in this
function.

Link:
http://events.linuxfoundation.org/sites/events/files/slides/LCJ16-R
efactor_Strings-WSang_0.pdf

Signed-off-by: Markus Elfring <redacted>
The patch is fine but the link in the commit message is irrelevant.
Please remove it.
(Yes, I read through the whole presentation to verify that. It was fun,
even.)
Agree.

Except this patch assumes the superfluous braces would be removed in
4/5. But it turns out that other patch must be dropped. It would have
been better to remove the braces in this patch.


Paul Bolle
Thread (45 messages) 45 messages, 10 authors, 2016-09-28
Keyboard shortcuts
hback out one level
jnext message in thread / next series-diff position
kprevious message in thread / previous series-diff position
ldrill in
Escclose help / fold thread tree
?toggle this help