Attention is currently required from: neels.
pespin has posted comments on this change. (
https://gerrit.osmocom.org/c/osmo-upf/+/27218
)
Change subject: libosmo-tlv: add auto dec/enc to/from structs
......................................................................
Patch Set 4:
(1 comment)
File include/osmocom/tlv/tlv_dec_enc.h:
https://gerrit.osmocom.org/c/osmo-upf/+/27218/comment/91a78070_e8014cbb
PS1, Line 135: * In this example, the nested IEs decode/encode to different goo sub
structs depending on the tag value.
well, we often use foo, bar, baz, but i also like to
use foo, goo, moo etc. […]
That sounds like the BORKEN state which already raised
confusion several times. I'd really avoid "goo" and use "foo"
instead, or simiply drop "goo".
--
To view, visit
https://gerrit.osmocom.org/c/osmo-upf/+/27218
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-upf
Gerrit-Branch: master
Gerrit-Change-Id: I65de793105882a452124ee58adb0e58469e6e796
Gerrit-Change-Number: 27218
Gerrit-PatchSet: 4
Gerrit-Owner: neels <nhofmeyr(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: neels <nhofmeyr(a)sysmocom.de>
Gerrit-Comment-Date: Thu, 17 Mar 2022 15:44:28 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: neels <nhofmeyr(a)sysmocom.de>
Comment-In-Reply-To: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: comment