This is merely a historical archive of years 2008-2021, before the migration to mailman3.
A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.
neels gerrit-no-reply at lists.osmocom.orgneels has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-msc/+/18692 ) Change subject: msc_a: add callref as call id to ASSIGNMENT REQ. ...................................................................... Patch Set 2: Code-Review-1 (2 comments) (would be nice to also see the code that is using this new IE in the same patch, or mention in the commit log the change id of the patch that will add that later) https://gerrit.osmocom.org/c/osmo-msc/+/18692/2/include/osmocom/msc/ran_msg.h File include/osmocom/msc/ran_msg.h: https://gerrit.osmocom.org/c/osmo-msc/+/18692/2/include/osmocom/msc/ran_msg.h@89 PS2, Line 89: uint32_t call_id; each of these structs must allow indicating presence or absence of optional IEs. The earlier patch set using call_id_present was exactly the right thing to do. sorry to add another iteration, but please bring back call_id_present. (The idea of these structs is to represent 1:1 what the encoded message contained/should contain, and not make any assumptions on how individual IEs interact.) https://gerrit.osmocom.org/c/osmo-msc/+/18692/2/src/libmsc/ran_msg_a.c File src/libmsc/ran_msg_a.c: https://gerrit.osmocom.org/c/osmo-msc/+/18692/2/src/libmsc/ran_msg_a.c@998 PS2, Line 998: call_id = &ac->call_id; I don't understand why the call_id is related to RTP address. I prefer the earlier patch set handling the call_id separately by call_id_present. -- To view, visit https://gerrit.osmocom.org/c/osmo-msc/+/18692 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-msc Gerrit-Branch: master Gerrit-Change-Id: I4288f47e4a6d61ec672f431723f6e72c7c6b0799 Gerrit-Change-Number: 18692 Gerrit-PatchSet: 2 Gerrit-Owner: dexter <pmaier at sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: dexter <pmaier at sysmocom.de> Gerrit-Reviewer: neels <nhofmeyr at sysmocom.de> Gerrit-Reviewer: pespin <pespin at sysmocom.de> Gerrit-CC: laforge <laforge at osmocom.org> Gerrit-Comment-Date: Sun, 14 Jun 2020 12:10:46 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes Gerrit-MessageType: comment -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20200614/55bb1f63/attachment.htm>