keith has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-sip-connector/+/15019 )

Change subject: Transmit Bearer Caps to MNCC with SETUP_REQ
......................................................................


Patch Set 1:

(1 comment)

https://gerrit.osmocom.org/#/c/15019/1/src/mncc.c
File src/mncc.c:

https://gerrit.osmocom.org/#/c/15019/1/src/mncc.c@824
PS1, Line 824: struct gsm_mncc_bearer_cap bearer_cap
> I think you can assign the new structure directly to mncc. […]
Right, I started thinking to set a default bearer cap in a temporary variable 
that would apply for all codecs, then only change what is dependent on the 
codec. finally assign it to mncc.bearer_cap. it would avoid duplication. It 
just ended up this way for now.

I'm not sure which fields actually make any difference to the MSC.



--
To view, visit https://gerrit.osmocom.org/c/osmo-sip-connector/+/15019
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-sip-connector
Gerrit-Branch: master
Gerrit-Change-Id: I55cdc4b7c52730d7e998a1d692c7552fd07403ee
Gerrit-Change-Number: 15019
Gerrit-PatchSet: 1
Gerrit-Owner: keith <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: keith <[email protected]>
Gerrit-CC: fixeria <[email protected]>
Gerrit-Comment-Date: Thu, 01 Aug 2019 22:39:53 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: fixeria <[email protected]>
Gerrit-MessageType: comment

Reply via email to