[asterisk-dev] [Code Review] 3862: testsuite: run-local creates broken symlinks for tests

--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3862/ --- Review request for Asterisk Developers. Bugs: ASTERISK-24126 https://i

Re: [asterisk-dev] [Code Review] 3859: datastores: Audit v1.8 ast_channel_datastore_remove usage.

--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3859/ --- (Updated July 25, 2014, 9:04 p.m.) Review request for Asterisk Developers.

Re: [asterisk-dev] [Code Review] 3860: datastores: Audit v11 ast_channel_datastore_remove usage.

--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3860/ --- (Updated July 25, 2014, 9:03 p.m.) Review request for Asterisk Developers.

[asterisk-dev] [Code Review] 3861: datastores: Audit v12 ast_channel_datastore_remove usage.

--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3861/ --- Review request for Asterisk Developers. Repository: Asterisk Description

[asterisk-dev] [Code Review] 3860: datastores: Audit v11 ast_channel_datastore_remove usage.

--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3860/ --- Review request for Asterisk Developers. Repository: Asterisk Description

Re: [asterisk-dev] [Code Review] 3844: features.c: Allow appliationmap to use Gosub.

--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3844/ --- (Updated July 25, 2014, 6:04 p.m.) Status -- This change has been mar

Re: [asterisk-dev] [Code Review] 3673: RLS: Nominal list tests

--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3673/ --- (Updated July 25, 2014, 10:54 p.m.) Review request for Asterisk Developers

Re: [asterisk-dev] [Code Review] 3859: datastores: Audit v1.8 ast_channel_datastore_remove usage.

--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3859/ --- (Updated July 25, 2014, 5:23 p.m.) Review request for Asterisk Developers.

[asterisk-dev] [Code Review] 3859: datastores: Audit v1.8 ast_channel_datastore_remove usage.

--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3859/ --- Review request for Asterisk Developers. Repository: Asterisk Description

[asterisk-dev] Planned maintenance for community services on Monday, July 28, 2014

Monday, July 28, 2014, Asterisk community services will have intermittent availability due to maintenance. This maintenance will begin at approximately 9:00 PM CST[1] and should last no longer than one hour. The affected services are: * svn.digium.com * svn.asterisk.org * svncommunity.digium.com

Re: [asterisk-dev] [Code Review] 3852: Stasis App: Handle outbound masquerades

--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3852/ --- (Updated July 25, 2014, 5:15 p.m.) Status -- This change has been mar

Re: [asterisk-dev] [Code Review] 3741: RLS: Add body generation + some bug fixes

--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3741/ --- (Updated July 25, 2014, 10 p.m.) Review request for Asterisk Developers.

Re: [asterisk-dev] [Code Review] 3852: Stasis App: Handle outbound masquerades

--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3852/#review12880 --- Ship it! Minor nit. team/group/ari-greedy-atxfer/res/res_sta

Re: [asterisk-dev] [Code Review] 3852: Stasis App: Handle outbound masquerades

--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3852/ --- (Updated July 25, 2014, 4:30 p.m.) Review request for Asterisk Developers,

Re: [asterisk-dev] [Code Review] 3723: RLS: NOTIFY generation + structural refactor

--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3723/ --- (Updated July 25, 2014, 8:57 p.m.) Review request for Asterisk Developers.

Re: [asterisk-dev] [Code Review] 3699: RLS: Inbound SUBSCRIBE handling

--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3699/ --- (Updated July 25, 2014, 8:46 p.m.) Status -- This change has been mar

Re: [asterisk-dev] [Code Review] 3852: Stasis App: Handle outbound masquerades

--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3852/#review12878 --- team/group/ari-greedy-atxfer/res/res_stasis.c

[asterisk-dev] [Code Review] 3858: Tests for verifying StopMixMonitor/MuteMixMonitor AMI action

--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3858/ --- Review request for Asterisk Developers. Bugs: ASTERISK-24028 https://i

Re: [asterisk-dev] [Code Review] 3857: Ensure that channels that move from a stasis bridge to a non-stasis bridge have a StasisEnd sent at an appropriate time.

--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3857/#review12879 --- /team/group/ari-greedy-atxfer/include/asterisk/bridge_channel.

[asterisk-dev] [Code Review] 3857: Ensure that channels that move from a stasis bridge to a non-stasis bridge have a StasisEnd sent at an appropriate time.

--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3857/ --- Review request for Asterisk Developers. Repository: Asterisk Description

Re: [asterisk-dev] [Code Review] 3795: Test for MixMonitor Recording Feature

--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3795/ --- (Updated July 25, 2014, 2:51 p.m.) Review request for Asterisk Developers.

Re: [asterisk-dev] [Code Review] 3795: Test for MixMonitor Recording Feature

--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3795/ --- (Updated July 25, 2014, 2:50 p.m.) Review request for Asterisk Developers.

Re: [asterisk-dev] [Code Review] 3716: Weak Reference Containers

> On July 24, 2014, 4:01 p.m., rmudgett wrote: > > You are affecting the lifetime of the continer node object without > > protecting it. It can now potentially be used outside the lifetime of its > > container as part of the held object. The container node and the continer > > ponter in the

Re: [asterisk-dev] [Code Review] 3716: Weak Reference Containers

> On July 24, 2014, 3:01 p.m., rmudgett wrote: > > You are affecting the lifetime of the continer node object without > > protecting it. It can now potentially be used outside the lifetime of its > > container as part of the held object. The container node and the continer > > ponter in the

Re: [asterisk-dev] [Code Review] 3839: Added a test to check the duration and energy_duration values of an ARI recording

--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3839/ --- (Updated July 25, 2014, 5:44 p.m.) Review request for Asterisk Developers.

Re: [asterisk-dev] [Code Review] 3733: Added a pluggable module for pluggable_modules.py that allows the user to test different attributes of a given sound file

--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3733/ --- (Updated July 25, 2014, 5:41 p.m.) Review request for Asterisk Developers.

Re: [asterisk-dev] [Code Review] 3716: Weak Reference Containers

> On July 24, 2014, 4:01 p.m., rmudgett wrote: > > You are affecting the lifetime of the continer node object without > > protecting it. It can now potentially be used outside the lifetime of its > > container as part of the held object. The container node and the continer > > ponter in the

Re: [asterisk-dev] [Code Review] 3733: Added a pluggable module for pluggable_modules.py that allows the user to test different attributes of a given sound file

--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3733/#review12874 --- Ship it! Ship It! - Matt Jordan On July 24, 2014, 12:08 p.m

Re: [asterisk-dev] [Code Review] 3716: Weak Reference Containers

> On July 24, 2014, 3:01 p.m., rmudgett wrote: > > You are affecting the lifetime of the continer node object without > > protecting it. It can now potentially be used outside the lifetime of its > > container as part of the held object. The container node and the continer > > ponter in the

Re: [asterisk-dev] [Code Review] 3716: Weak Reference Containers

> On July 24, 2014, 3:01 p.m., rmudgett wrote: > > You are affecting the lifetime of the continer node object without > > protecting it. It can now potentially be used outside the lifetime of its > > container as part of the held object. The container node and the continer > > ponter in the

Re: [asterisk-dev] [Code Review] 3802: Add module support level to module info; print module support level in CLI "module show" command.

--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3802/ --- (Updated July 25, 2014, 11:47 a.m.) Status -- This change has been ma

Re: [asterisk-dev] [Code Review] 3716: Weak Reference Containers

> On July 24, 2014, 4:01 p.m., rmudgett wrote: > > You are affecting the lifetime of the continer node object without > > protecting it. It can now potentially be used outside the lifetime of its > > container as part of the held object. The container node and the continer > > ponter in the

Re: [asterisk-dev] [Code Review] 3716: Weak Reference Containers

> On July 24, 2014, 3:01 p.m., rmudgett wrote: > > You are affecting the lifetime of the continer node object without > > protecting it. It can now potentially be used outside the lifetime of its > > container as part of the held object. The container node and the continer > > ponter in the

[asterisk-dev] [Code Review] 3856: Fix Dynamic IAX2 Registrations From Failing Or Not Retrying After Temporary DNS Failure

--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3856/ --- Review request for Asterisk Developers. Bugs: ASTERISK-23767 https://i

Re: [asterisk-dev] [Code Review] 3770: ARI: report duration values in LiveRecording objects

--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3770/ --- (Updated July 25, 2014, 9:41 a.m.) Status -- This change has been mar

Re: [asterisk-dev] [Code Review] 3777: module loader: Unload modules in reverse order of their start order

--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3777/ --- (Updated July 25, 2014, 9:27 a.m.) Status -- This change has been mar

Re: [asterisk-dev] [Code Review] 3854: manager.c - Improve documentation for manager command Getvar, Setvar

--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3854/ --- (Updated July 25, 2014, 2:20 p.m.) Review request for Asterisk Developers.

Re: [asterisk-dev] [Code Review] 3854: manager.c - Improve documentation for manager command Getvar, Setvar

--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3854/ --- (Updated July 25, 2014, 2:19 p.m.) Review request for Asterisk Developers.

[asterisk-dev] [Code Review] 3854: manager.c - Improve documentation for manager command Getvar, Setvar

--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3854/ --- Review request for Asterisk Developers. Bugs: ASTERISK-21178 https://i

[asterisk-dev] [Code Review] 3852: Stasis App: Handle outbound masquerades

--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3852/ --- Review request for Asterisk Developers, Mark Michelson and rmudgett. Bugs:

[asterisk-dev] [Code Review] 3851: res_pjsip_pidf_body_generator / res_pjsip_xpidf_body_generator: Ensure entity is unquoted.

--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3851/ --- Review request for Asterisk Developers. Repository: Asterisk Description

Re: [asterisk-dev] [Code Review] 3844: features.c: Allow appliationmap to use Gosub.

--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3844/#review12869 --- Ship it! /branches/1.8/apps/app_stack.c

[asterisk-dev] [Code Review] 3850: res_pjsip_session: Ensure session supplements are invoked before redirection occurs

--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3850/ --- Review request for Asterisk Developers. Repository: Asterisk Description

Re: [asterisk-dev] [Code Review] 3836: app_bridgewait: Remove race condition where bridge may be dissolved when trying to join

--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3836/ --- (Updated July 25, 2014, 5:53 a.m.) Status -- This change has been mar

Re: [asterisk-dev] [Code Review] 3840: bridge: Move "bridge destroy" CLI command to devmode and add "all" to "bridge kick" CLI command

--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3840/ --- (Updated July 25, 2014, 5:49 a.m.) Status -- This change has been mar

Re: [asterisk-dev] [Code Review] 3844: features.c: Allow appliationmap to use Gosub.

--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3844/#review12868 --- Ship it! Tested with Asterisk 11: Gosub is now working as expe

Re: [asterisk-dev] [svn-commits] mjordan: trunk r419358 - in /trunk: channels/chan_pjsip.c main/devicestate.c

On 24 Jul 2014, at 17:21, SVN commits to the Digium repositories wrote: > device state: Update the core to report ONHOLD if a channel is on hold > Aren't you mixing channel states with device and extension states here? A phone (a device or line) can't be on hold - a call can be. It's an attri