Attention is currently required from: falconia.

Jenkins Builder has posted comments on this change by falconia. ( 
https://gerrit.osmocom.org/c/libosmo-abis/+/38139?usp=email )

Change subject: tests: add unit tests for trau2rtp conversion, FR & EFR
......................................................................


Patch Set 3:

(6 comments)

File tests/trau_conv/trau16_to_rtp.c:

Robot Comment from checkpatch (run ID jenkins-gerrit-lint-18531):
https://gerrit.osmocom.org/c/libosmo-abis/+/38139/comment/31eb183a_b1bb2c54?usp=email
 :
PS3, Line 196:          /* The only valid 1-byte payload is TEH by itself
'TEH' may be misspelled - perhaps 'THE'?


Robot Comment from checkpatch (run ID jenkins-gerrit-lint-18531):
https://gerrit.osmocom.org/c/libosmo-abis/+/38139/comment/57458703_59a822ba?usp=email
 :
PS3, Line 200:          printf("  TW-TS-001 TEH octet: 0x%02X\n", rtp_pl[0]);
'TEH' may be misspelled - perhaps 'THE'?


Robot Comment from checkpatch (run ID jenkins-gerrit-lint-18531):
https://gerrit.osmocom.org/c/libosmo-abis/+/38139/comment/7bd38afd_40a6e1d5?usp=email
 :
PS3, Line 213:          printf("  TW-TS-001 TEH octet: 0x%02X\n", rtp_pl[0]);
'TEH' may be misspelled - perhaps 'THE'?


Robot Comment from checkpatch (run ID jenkins-gerrit-lint-18531):
https://gerrit.osmocom.org/c/libosmo-abis/+/38139/comment/5d4b9326_78fbb185?usp=email
 :
PS3, Line 250:          /* The only valid 1-byte payload is TEH by itself
'TEH' may be misspelled - perhaps 'THE'?


Robot Comment from checkpatch (run ID jenkins-gerrit-lint-18531):
https://gerrit.osmocom.org/c/libosmo-abis/+/38139/comment/55ce8998_87e01bec?usp=email
 :
PS3, Line 254:          printf("  TW-TS-001 TEH octet: 0x%02X\n", rtp_pl[0]);
'TEH' may be misspelled - perhaps 'THE'?


Robot Comment from checkpatch (run ID jenkins-gerrit-lint-18531):
https://gerrit.osmocom.org/c/libosmo-abis/+/38139/comment/a9790a0e_18f8e9e8?usp=email
 :
PS3, Line 267:          printf("  TW-TS-001 TEH octet: 0x%02X\n", rtp_pl[0]);
'TEH' may be misspelled - perhaps 'THE'?



--
To view, visit https://gerrit.osmocom.org/c/libosmo-abis/+/38139?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings?usp=email

Gerrit-MessageType: comment
Gerrit-Project: libosmo-abis
Gerrit-Branch: master
Gerrit-Change-Id: Ie319900125baa8f20595cac83343771d780a5b4e
Gerrit-Change-Number: 38139
Gerrit-PatchSet: 3
Gerrit-Owner: falconia <fal...@freecalypso.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Attention: falconia <fal...@freecalypso.org>
Gerrit-Comment-Date: Sat, 14 Sep 2024 07:53:48 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No

Reply via email to