Attention is currently required from: matanp.

pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/gapk/+/32094 )

Change subject: fmt_rtp_amr,fmt_rtp_efr: replace damaged packets with silence
......................................................................


Patch Set 3: Code-Review+1

(1 comment)

File src/fmt_rtp_efr.c:

https://gerrit.osmocom.org/c/gapk/+/32094/comment/ad5b6b3e_5d09ebe5
PS2, Line 70:   if (!memcmp(DAMAGED_PACKET, src, EFR_LEN))
> Yes, the if statement in line 65 checks it
Ack



--
To view, visit https://gerrit.osmocom.org/c/gapk/+/32094
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: gapk
Gerrit-Branch: master
Gerrit-Change-Id: I7245aa0bc0955cc8b94d5401a15e694f50498093
Gerrit-Change-Number: 32094
Gerrit-PatchSet: 3
Gerrit-Owner: matanp <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin <[email protected]>
Gerrit-CC: laforge <[email protected]>
Gerrit-Attention: matanp <[email protected]>
Gerrit-Comment-Date: Tue, 28 Mar 2023 15:30:55 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Comment-In-Reply-To: pespin <[email protected]>
Comment-In-Reply-To: matanp <[email protected]>
Gerrit-MessageType: comment

Reply via email to