Attention is currently required from: osmith.
pespin has posted comments on this change. (
https://gerrit.osmocom.org/c/osmo-iuh/+/30088
)
Change subject: asn1tostruct: fix defines getting redefined
......................................................................
Patch Set 3: Code-Review+1
(1 comment)
Patchset:
PS3:
Yes. Here is the diff of the generated files with this
patch:
https://gitea.osmocom. […]
ACK. IIUC you didn't detect any code change
required after this change in the users of the lib? In any case it may make sense to apply
the renaming only for fields count > 1 (so that if there's 2 or more fields
colliding, the first one is kept the old way).
--
To view, visit
https://gerrit.osmocom.org/c/osmo-iuh/+/30088
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-iuh
Gerrit-Branch: master
Gerrit-Change-Id: I2ecae6789899952d1dc5691ab76907abeaa71c12
Gerrit-Change-Number: 30088
Gerrit-PatchSet: 3
Gerrit-Owner: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: osmith <osmith(a)sysmocom.de>
Gerrit-Comment-Date: Mon, 14 Nov 2022 15:09:12 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Comment-In-Reply-To: osmith <osmith(a)sysmocom.de>
Comment-In-Reply-To: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: comment