Attention is currently required from: falconia, pespin.
fixeria has posted comments on this change. (
https://gerrit.osmocom.org/c/libosmo-abis/+/37287?usp=email )
Change subject: trau_rtp_conv: add support for HRv1 in 8k format
......................................................................
Patch Set 2: Code-Review+1
(1 comment)
File src/trau/trau_rtp_conv.c:
https://gerrit.osmocom.org/c/libosmo-abis/+/37287/comment/20a3b2d6_b5d058c5
PS2, Line 669: tf->xc_bits[0] = 0;
I tried doing this: […]
Oh, then my approach
would not work either. AFAIU, array assignment is only possible during the definition, but
not after that. I did an experiment, and even with `-std=c89` both gcc and clang refuse to
compile the following code:
```
int main(void)
{
char data[6];
data = { 0, 0, 0, 1, 0, 0 };
data = { [3] = 1 };
return 0;
}
```
So yeah, let's keep this as-is then.
--
To view, visit
https://gerrit.osmocom.org/c/libosmo-abis/+/37287?usp=email
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: libosmo-abis
Gerrit-Branch: master
Gerrit-Change-Id: I8ee01b73360501ca380a8695cbc7070ceaaba1be
Gerrit-Change-Number: 37287
Gerrit-PatchSet: 2
Gerrit-Owner: falconia <falcon(a)freecalypso.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: falconia <falcon(a)freecalypso.org>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Tue, 25 Jun 2024 18:41:31 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Comment-In-Reply-To: falconia <falcon(a)freecalypso.org>
Comment-In-Reply-To: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-MessageType: comment