[Asterisk-code-review] codec negotiation: add incoming_call_offer_prefs option (asterisk[master])
Kevin Harwell
asteriskteam at digium.com
Thu Feb 27 16:45:53 CST 2020
Kevin Harwell has posted comments on this change. ( https://gerrit.asterisk.org/c/asterisk/+/13842 )
Change subject: codec negotiation: add incoming_call_offer_prefs option
......................................................................
Patch Set 2:
(1 comment)
https://gerrit.asterisk.org/c/asterisk/+/13842/1/include/asterisk/res_pjsip_session_caps.h
File include/asterisk/res_pjsip_session_caps.h:
https://gerrit.asterisk.org/c/asterisk/+/13842/1/include/asterisk/res_pjsip_session_caps.h@61
PS1, Line 61: * Creates a set of joint capabilities between the given remote capabilities,
: * and pre-configured ones.
> This API feels weird, because generally these things return a unique capabilities structure that isn […]
I understand, but I think "update" is wrong here as well since we're not really updating it (even won't update if one is already present). I went with "join" in hopes that that is less confusing? Naming is hard!
--
To view, visit https://gerrit.asterisk.org/c/asterisk/+/13842
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Change-Id: I35e7a2a0c236cfb6bd9cdf89539f57a1ffefc76f
Gerrit-Change-Number: 13842
Gerrit-PatchSet: 2
Gerrit-Owner: Kevin Harwell <kharwell at digium.com>
Gerrit-Reviewer: Friendly Automation
Gerrit-Reviewer: Joshua Colp <jcolp at sangoma.com>
Gerrit-CC: Sean Bright <sean.bright at gmail.com>
Gerrit-Comment-Date: Thu, 27 Feb 2020 22:45:53 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Joshua Colp <jcolp at sangoma.com>
Gerrit-MessageType: comment
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20200227/8e7cceb0/attachment.html>
More information about the asterisk-code-review
mailing list