Attention is currently required from: laforge.
6 comments:
File contrib/ber/rtp_ber.c:
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-5283):
Patch Set #5, 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-5283):
Patch Set #5, 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-5283):
Patch Set #5, Line 480: while (1) {
braces {} are not necessary for single statement blocks
File contrib/ber/rtp_gen_map.c:
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-5283):
Patch Set #5, Line 34: for (i = 0; i < 260; i++)
that open brace { should be on the previous line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-5283):
Patch Set #5, Line 55: for (i = 0; i < 260; i++)
that open brace { should be on the previous line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-5283):
Patch Set #5, Line 116: for (i = 0; i < len; i++)
that open brace { should be on the previous line
To view, visit change 31967. To unsubscribe, or for help writing mail filters, visit settings.