Attention is currently required from: fixeria, pespin. neels has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/35053?usp=email )
Change subject: msc: test re-assignment to match codec with MT ...................................................................... Patch Set 2: (3 comments) File msc/BSC_ConnectionHandler.ttcn: https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/35053/comment/3f87abb7_e977454f PS2, Line 986: /* What to Codec (Chosen) to send back in Assignment Complete */ > "What to Codec to" doesn't make sense to me? […] thx https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/35053/comment/c7e7b22a_e3a1695c PS2, Line 993: BSSMAP_FIELD_CodecElements re_ass_req_codecs, > Sounds like "optional" is missing in this field. it is always {FR} by default, so it is never actually absent. Should it still be marked optional? There is no practical benefit of 'optional', because there already is the expect_re_assignment flag -- that bool indicates usage for both re_ass_req_codecs and re_ass_compl_chosen_codec, so adding more optionality makes checks for presence more complex; point being that there is no caller ever using the optional part. The same is true for the already existing transaction_id: it is marked optional, yet it is always initialized. So maybe I am wrong here? But why add more possible states == more complexity?? https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/35053/comment/664a4c15_0aaadc61 PS2, Line 995: re_ass_compl_chosen_codec > Sounds like "optional" is missing in this field. (s.a.) -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/35053?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-ttcn3-hacks Gerrit-Branch: master Gerrit-Change-Id: I402ed0523a2a87b83f29c5577b2c828102005d53 Gerrit-Change-Number: 35053 Gerrit-PatchSet: 2 Gerrit-Owner: neels <[email protected]> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <[email protected]> Gerrit-Reviewer: pespin <[email protected]> Gerrit-Attention: pespin <[email protected]> Gerrit-Attention: fixeria <[email protected]> Gerrit-Comment-Date: Sat, 15 Jun 2024 03:06:46 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: neels <[email protected]> Comment-In-Reply-To: pespin <[email protected]> Comment-In-Reply-To: fixeria <[email protected]> Gerrit-MessageType: comment
