Attention is currently required from: falconia.
pespin has posted comments on this change by falconia. ( https://gerrit.osmocom.org/c/libosmo-netif/+/39281?usp=email )
Change subject: bring twrtp into libosmo-netif ......................................................................
Patch Set 6:
(2 comments)
File include/osmocom/netif/twrtp.h:
https://gerrit.osmocom.org/c/libosmo-netif/+/39281/comment/7cccd5fc_eec680cb... : PS1, Line 26: struct osmo_twrtp_stats { Similarly to the public struct discussion I opened in the previous twjit patch, it also applies here.
File src/twrtp_base.c:
https://gerrit.osmocom.org/c/libosmo-netif/+/39281/comment/f804d619_c85a0076... : PS1, Line 132: * another variant of this API with that alternative behavior.
Patchset 6 includes an implementation of this requested feature, making RTCP UDP socket optional. […]
I'd prefer deferr/avoid allocating rtcp iofd if not needed.