Attention is currently required from: laforge.
Jenkins Builder has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bts/+/31967 )
Change subject: contrib: Add BER testing tool ......................................................................
Patch Set 6:
(3 comments)
File contrib/ber/rtp_ber.c:
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-5284): https://gerrit.osmocom.org/c/osmo-bts/+/31967/comment/0c83f43d_281d79ab PS6, Line 97: static const char *amr_rate_by_ft[] = { static const char * array should probably be static const char * const
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-5284): https://gerrit.osmocom.org/c/osmo-bts/+/31967/comment/d81decbe_9fd0ebc3 PS6, Line 416: if (rtp_seq_num_diff(seq_number, as->rx_last_seq) > 1) { braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-5284): https://gerrit.osmocom.org/c/osmo-bts/+/31967/comment/843578fd_551e6fbd PS6, Line 480: while (1) { braces {} are not necessary for single statement blocks