daniel has posted comments on this change. ( https://gerrit.osmocom.org/c/libosmocore/+/32181 )

Change subject: tests: Add initial osmo_io tests ......................................................................

Patch Set 2:

(1 comment)

File tests/osmo_io/osmo_io_test.c:

https://gerrit.osmocom.org/c/libosmocore/+/32181/comment/7d02f4c7_cc8e421a PS1, Line 144: for (int i = 0; i < 128; i++) {

braces {} are not necessary for single statement blocks

tbh I'm really not a fan of this rule since it's really easy to get bitten by it during a refactoring.

At least there are now warnings when the indentation level doesn't match the assumed control flow...