RE: [PATCH next 10/11] block: Use a boolean expression instead of max() on booleans

On Mon, 29 Jan 2024, David Laight [off-list ref] wrote:

From: Jani Nikula

quoted

Sent: 29 January 2024 09:08

On Sun, 28 Jan 2024, David Laight [off-list ref] wrote:

quoted

blk_stack_limits() contains:
	t->zoned = max(t->zoned, b->zoned);
These are bool, so it is just a bitwise or.

Should be a logical or, really. And || in code.

Not really, bitwise is fine for bool (especially for 'or')
and generates better code.

Logical operations for booleans are more readable for humans than
bitwise. And semantically correct.

With a = b || c you know what happens regardless of the types in
question. a = b | c you have to look up the types to know what's going
on.

To me, better code only matters if it's a hotpath.

That said, not my are of maintenance, so *shrug*.


BR,
Jani.


-- 
Jani Nikula, Intel