[Asterisk-code-review] chan_sip, chan_iax2, chan_dahdi: Allow updating variable value. (...asterisk[master])

Guido Falsi asteriskteam at digium.com
Fri May 10 08:16:02 CDT 2019


Hello Benjamin Keith Ford, Friendly Automation, George Joseph, Sean Bright, Joshua Colp, 

I'd like you to reexamine a change. Please visit

    https://gerrit.asterisk.org/c/asterisk/+/10930

to look at the new patch set (#12).

Change subject: chan_sip, chan_iax2, chan_dahdi: Allow updating variable value.
......................................................................

chan_sip, chan_iax2, chan_dahdi: Allow updating variable value.

When modifying an already defined variable in chan_sip and chan_iax2
drivers they add a new variable with the same name to the linked
list, but that value is never used, only the first one found. This
patch makes the code replace the old value with the new one.
Document added support for setvar in chan_iax2.

ASTERISK-23756
patches:
  setvar-multiplie.patch submitted by Michael Goryainov

Change-Id: Ie1897a96c82b8945e752733612ee963686f32839
---
M channels/chan_dahdi.c
M channels/chan_iax2.c
M channels/chan_sip.c
M configs/samples/iax.conf.sample
A doc/CHANGES-staging/channel_variable_update_value.txt
M include/asterisk/config.h
M main/config.c
M res/res_pjsip/pjsip_configuration.c
8 files changed, 84 insertions(+), 28 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/30/10930/12
-- 
To view, visit https://gerrit.asterisk.org/c/asterisk/+/10930
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Change-Id: Ie1897a96c82b8945e752733612ee963686f32839
Gerrit-Change-Number: 10930
Gerrit-PatchSet: 12
Gerrit-Owner: Guido Falsi <madpilot at freebsd.org>
Gerrit-Reviewer: Benjamin Keith Ford <bford at digium.com>
Gerrit-Reviewer: Friendly Automation
Gerrit-Reviewer: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Guido Falsi <madpilot at freebsd.org>
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Sean Bright <sean.bright at gmail.com>
Gerrit-MessageType: newpatchset
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20190510/b1918427/attachment.html>


More information about the asterisk-code-review mailing list