Attention is currently required from: falconia, fixeria, jolly, laforge. pespin has posted comments on this change by falconia. ( https://gerrit.osmocom.org/c/libosmo-netif/+/39280?usp=email )
Change subject: bring twjit into libosmo-netif ...................................................................... Patch Set 6: (2 comments) File include/osmocom/netif/twjit.h: https://gerrit.osmocom.org/c/libosmo-netif/+/39280/comment/23fdb85b_284f0daf?usp=email : PS6, Line 50: struct osmo_twjit_config { > The only struct that poses truly serious ABI issues is `osmo_twjit_config`. > […] @fal...@freecalypso.org my preferred solution would be to actually remove the osmo_twjit_config from public space and instead have setter API for each param one wants to configure on the twjit. Only pass the really required ones during osmo_twjit_alloc(), and for the others set a sane default plus the mentioned setters to configure/tweak it afterwards. I know at first hand it looks like more work and more function calls to be made, but in the end this allows good maintainability, extensibility, etc. I'm saying this based on experience, for instance see the osmo_stream (stream.h) API which follows this approach and proved to cope well with extensions over time. Your proposal would work and I could accept it, but it I really dislike the idea of having to allocate an extra twjit_config struct. Let me know if my proposal is good for you. File src/twjit.c: https://gerrit.osmocom.org/c/libosmo-netif/+/39280/comment/9c3f6c13_183cabc3?usp=email : PS2, Line 504: rtph = osmo_rtp_get_hdr(msg); > Can this issue be marked as resolved - is M bit handled to your satisfaction > now? Done -- To view, visit https://gerrit.osmocom.org/c/libosmo-netif/+/39280?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: libosmo-netif Gerrit-Branch: master Gerrit-Change-Id: Ia3be5834571ca18b68939abbcf1ce3a879156658 Gerrit-Change-Number: 39280 Gerrit-PatchSet: 6 Gerrit-Owner: falconia <fal...@freecalypso.org> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanits...@sysmocom.de> Gerrit-Reviewer: jolly <andr...@eversberg.eu> Gerrit-Reviewer: laforge <lafo...@osmocom.org> Gerrit-Reviewer: pespin <pes...@sysmocom.de> Gerrit-Attention: jolly <andr...@eversberg.eu> Gerrit-Attention: falconia <fal...@freecalypso.org> Gerrit-Attention: laforge <lafo...@osmocom.org> Gerrit-Attention: fixeria <vyanits...@sysmocom.de> Gerrit-Comment-Date: Tue, 26 Aug 2025 10:15:43 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: falconia <fal...@freecalypso.org> Comment-In-Reply-To: laforge <lafo...@osmocom.org> Comment-In-Reply-To: pespin <pes...@sysmocom.de>
