[asterisk-dev] [Code Review] 3129: chan_pjsip: Provide a means for updating device state when going on/off hold

[Jonathan Rose]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/3129/#review10609
-----------------------------------------------------------



/branches/12/channels/chan_pjsip.c
<https://reviewboard.asterisk.org/r/3129/#comment20106>

    I used a magic number here because I couldn't find an appropriate definition for device name lengths. If one is known, I'll be happy to update this.



/branches/12/channels/chan_pjsip.c
<https://reviewboard.asterisk.org/r/3129/#comment20108>

    Not really sure how necessary the RWLOCK is for this container. It makes sense to use it since there are polling actions that could take place synchronously and modification actions that probably shouldn't, but I'm not sure.


- Jonathan Rose


On Jan. 16, 2014, 12:26 p.m., Jonathan Rose wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/3129/
> -----------------------------------------------------------
> 
> (Updated Jan. 16, 2014, 12:26 p.m.)
> 
> 
> Review request for Asterisk Developers and Mark Michelson.
> 
> 
> Repository: Asterisk
> 
> 
> Description
> -------
> 
> Maintains a list of on hold channel uniqueids.  When a snapshot is used with chan_pjsip's device state evaluating function, this list is queried to see if the channel in the snapshot is on hold. As the channel receives indications of HOLD/UNHOLD, this list is updated and the device state is updated. When the session ends, the session channel is removed from the list if it is still in it.
> 
> 
> Diffs
> -----
> 
>   /branches/12/channels/chan_pjsip.c 405641 
> 
> Diff: https://reviewboard.asterisk.org/r/3129/diff/
> 
> 
> Testing
> -------
> 
> Manual testing where I watched the device state with a websocket, polled for device state with Manager command GetVar and DEVICE_STATE function, and also modified an automated test (channels/pjsip/hold) to have a hint which checks the device state of a PJSIP channel and ExtensionStatus event is monitored to evaluate device state changes as they had.
> 
> The test modifications will be in another review.
> 
> 
> Thanks,
> 
> Jonathan Rose
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20140116/d41d724c/attachment-0001.html>