----------------------------------------------------------- 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 > >
-- _____________________________________________________________________ -- Bandwidth and Colocation Provided by http://www.api-digital.com -- asterisk-dev mailing list To UNSUBSCRIBE or update options visit: http://lists.digium.com/mailman/listinfo/asterisk-dev
