Attention is currently required from: pespin.
Patch set 1:Code-Review +1
View Change
1 comment:
File src/ss7_asp_vty.c:
Patch Set #1, Line 465: id64
Change it to id or asp_id?
Maybe bike-shedding, but the only relation to 64 is that the function takes an int64_t. Even id32 would make more sense to show that it only holds values in that range.
To view, visit change 42115. To unsubscribe, or for help writing mail filters, visit settings.
Gerrit-MessageType: comment
Gerrit-Project: libosmo-sigtran
Gerrit-Branch: master
Gerrit-Change-Id: Ib94d542f940e13d5c007bc3319e7dde65cf81f12
Gerrit-Change-Number: 42115
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: daniel <dwillmann@sysmocom.de>
Gerrit-Attention: pespin <pespin@sysmocom.de>
Gerrit-Comment-Date: Thu, 12 Feb 2026 15:31:19 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes