Re: [PATCH] staging: greybus: Reformat code in gb_operation_sync_timeout()
From: Thorsten Blum
Date: Mon Apr 14 2025 - 11:04:06 EST
On 14. Apr 2025, at 09:05, Johan Hovold wrote:
> This just makes the code *less* readable.
I guess you prefer the code with curly braces?
What about the Linux kernel coding style [1]? Specifically "Do not
unnecessarily use braces where a single statement will do."
My patch just removes any unnecessary curly braces, resulting in less
lines of code and no line break in the memcpy() arguments.
Thanks,
Thorsten
[1] https://docs.kernel.org/process/coding-style.html#placing-braces-and-spaces