Attention is currently required from: neels. laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-upf/+/27217 )
Change subject: libosmo-tlv: add versatile TLV de- and encoder ......................................................................
Patch Set 3: Code-Review-1
(1 comment)
Patchset:
PS3: as a general rule we don't introduce symbols or types with an osmo_ prefix into applications. This will likely clash with stuff from libraries.
So we can either move this library to libosmocore (or a separate library git repo), and keep the osmo_ prefix, or we can keep it locally in-tree and then remove/rename the osmo_ prefix.
If we move to libosmocore or a separate repo, we shuld also revisit the name. osmo_tlv and osmo_vtlv are already used with a different meaning in libosmocore. Maybe osmo_ntlv (new tlv)? ideas?