Messages in this thread | | | Date | Tue, 26 Sep 2017 15:43:16 +0900 | From | Sergey Senozhatsky <> | Subject | Re: [PATCH] printk: simplify no_printk() |
| |
On (09/18/17 00:01), Masahiro Yamada wrote: > Commit 069f0cd00df0 ("printk: Make the printk*once() variants return > a value") surrounded the macro implementation with ({ ... }). > > Now, the inner do { ... } while (0); is redundant. > > Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
looks ok to me. can't say that I see a huge difference, tho, but, FWIW:
Reviewed-by: Sergey Senozhatsky <sergey.senozhatsky@gmail.com>
-ss
| |