---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4606/
---
(Updated April 10, 2015, 4:04 p.m.)
Status
--
This change has been
On April 10, 2015, 9:50 a.m., Matt Jordan wrote:
I'm going to go ahead and post this for review on Gerrit
(gerrit.asterisk.org) - particularly since the 'findings' were very minor.
That will allow us to go ahead and commit the patch for the auto-dtmf mode
ahead of the move to Git
---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4609/#review15183
---
Reviewed: I didn't have any findings.
- Matt Jordan
On
---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4609/#review15184
---
Ship it!
/branches/13/main/channel.c
---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4108/#review15185
---
/trunk/main/astobj2.c
On April 10, 2015, 5:15 p.m., Mark Michelson wrote:
/branches/13/main/channel.c, lines 4146-4149
https://reviewboard.asterisk.org/r/4609/diff/1/?file=73842#file73842line4146
I figure I should point out why translate_via_slin is on by default.
The big reason is for audiohooks or
---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4609/
---
(Updated April 10, 2015, 11:30 p.m.)
Status
--
This change has been
On April 10, 2015, 7:05 p.m., rmudgett wrote:
/trunk/main/astobj2.c, lines 492-494
https://reviewboard.asterisk.org/r/4108/diff/5-6/?file=71780#file71780line492
The comment doesn't make sense. How is destructor_fn supposed to
access values in the weak proxy? The real object is
---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4585/
---
(Updated April 10, 2015, 6:34 p.m.)
Review request for Asterisk