Attention is currently required from: pespin.
fixeria has posted comments on this change. ( 
https://gerrit.osmocom.org/c/libosmocore/+/28120 )

Change subject: coding: cosmetic: move 'dtx_prev' to the scope where it's used
......................................................................


Patch Set 2:

(1 comment)

File src/coding/gsm0503_coding.c:

https://gerrit.osmocom.org/c/libosmocore/+/28120/comment/40699454_e0600888
PS1, Line 2186:                 const enum gsm0503_amr_dtx_frames dtx_prev = 
*dtx;
> I don't really see the  point in this patch. In any case, this should be at 
> the start of the block.
Done



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

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I147f44f1c071e53febeff425a0a7837a0ff10436
Gerrit-Change-Number: 28120
Gerrit-PatchSet: 2
Gerrit-Owner: fixeria <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: pespin <[email protected]>
Gerrit-Attention: pespin <[email protected]>
Gerrit-Comment-Date: Sun, 15 May 2022 22:13:35 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: pespin <[email protected]>
Gerrit-MessageType: comment

Reply via email to