Attention is currently required from: laforge, pespin, fixeria, daniel.
Jenkins Builder has posted comments on this change. ( https://gerrit.osmocom.org/c/libosmocore/+/32181 )
Change subject: tests: Add initial osmo_io tests ......................................................................
Patch Set 5:
(4 comments)
File tests/osmo_io/osmo_io_test.c:
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-6586): https://gerrit.osmocom.org/c/libosmocore/+/32181/comment/27b485ce_843b665e PS5, Line 54: printf("%s: Write returned rc=%d\n",osmo_iofd_get_name(iofd), rc); space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-6586): https://gerrit.osmocom.org/c/libosmocore/+/32181/comment/ce18be2e_e424cab8 PS5, Line 95: for (int i = 0; i < 128; i++) { braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-6586): https://gerrit.osmocom.org/c/libosmocore/+/32181/comment/db80d160_7e96c378 PS5, Line 146: for (int i = 0; i < 128; i++) { braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-6586): https://gerrit.osmocom.org/c/libosmocore/+/32181/comment/fb078ed3_f894b61b PS5, Line 174: } adding a line without newline at end of file