[greybus-dev] Re: [PATCH] staging: greybus: Reformat code in gb_operation_sync_timeout()

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