Hi,
I've just finished updating codec_opus for Asterisk 13. Unfortunately it
still requires a small patch to the core of Asterisk, but the size of
that patch is getting smaller with each new major version of Asterisk.
You can download the codec implementation and patch file here:
On Jan. 21, 2015, 5:31 p.m., Mark Michelson wrote:
/branches/13/cdr/cdr_amqp.c, lines 37-42
https://reviewboard.asterisk.org/r/4365/diff/1/?file=70953#file70953line37
This needs to have details filled in.
The actual doc is immediately below. This is an embarrassing vestigial
---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4327/
---
(Updated Jan. 26, 2015, 8:49 a.m.)
Status
--
This change has been
On 1/26/2015 8:16 AM, Olle E. Johansson wrote:
Hi,
I've just finished updating codec_opus for Asterisk 13. Unfortunately it still
requires a small patch to the core of Asterisk, but the size of that patch is
getting smaller with each new major version of Asterisk. You can download the
codec
---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4372/
---
Review request for Asterisk Developers and Jonathan Rose.
Bugs:
---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4365/
---
(Updated Jan. 26, 2015, 9:16 a.m.)
Review request for Asterisk
---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4373/
---
Review request for Asterisk Developers and jbigelow.
Repository: Asterisk
On 26 Jan 2015, at 14:03, Sean Bright sean.bri...@gmail.com wrote:
Hi,
I've just finished updating codec_opus for Asterisk 13. Unfortunately it
still requires a small patch to the core of Asterisk, but the size of that
patch is getting smaller with each new major version of Asterisk.
---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4368/#review14281
---
Ship it!
Ship It!
- Matt Jordan
On Jan. 23, 2015, 5:55
---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4374/
---
(Updated Jan. 26, 2015, 2:03 p.m.)
Review request for Asterisk
---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4367/#review14285
---
Ship it!
Ship It!
- Matt Jordan
On Jan. 26, 2015, 11:17
---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4373/#review14284
---
Also, this patch should be against Asterisk 13, not Asterisk
---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4374/
---
Review request for Asterisk Developers.
Bugs: ASTERISK-24316
---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4375/
---
Review request for Asterisk Developers.
Repository: Asterisk
---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4373/
---
(Updated Jan. 26, 2015, 2:16 p.m.)
Review request for Asterisk Developers
---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4369/#review14282
---
Ship it!
Ship It!
- Matt Jordan
On Jan. 23, 2015, 5:55
---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4373/#review14283
---
/branches/12/main/manager.c
---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4377/
---
Review request for Asterisk Developers.
Bugs: ASTERISK-24316
---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4367/
---
(Updated Jan. 26, 2015, 11:17 a.m.)
Review request for Asterisk
---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4373/#review14287
---
Ship it!
Ship It!
- Matt Jordan
On Jan. 26, 2015, 2:16
---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4375/#review14286
---
Ship it!
Ship It!
- Matt Jordan
On Jan. 26, 2015, 2:07
---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4377/
---
(Updated Jan. 26, 2015, 9:27 p.m.)
Review request for Asterisk
On Jan. 26, 2015, 5:44 p.m., rmudgett wrote:
./branches/13/main/http.c, line 2155
https://reviewboard.asterisk.org/r/4374/diff/1/?file=71085#file71085line2155
Setting a blank string will really mean a blank servername output value:
Server:
Is it intended for this
On Jan. 26, 2015, 5:44 p.m., rmudgett wrote:
./branches/13/main/http.c, line 2155
https://reviewboard.asterisk.org/r/4374/diff/1/?file=71085#file71085line2155
Setting a blank string will really mean a blank servername output value:
Server:
Is it intended for this
---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4372/
---
(Updated Jan. 26, 2015, 7:33 p.m.)
Review request for Asterisk Developers
On Jan. 26, 2015, 4:55 p.m., Scott Griepentrog wrote:
/branches/13/apps/confbridge/conf_config_parser.c, line 2304
https://reviewboard.asterisk.org/r/4372/diff/1/?file=71080#file71080line2304
Use ast_copy_string?
rmudgett wrote:
I second this should be ast_copy_string() for
---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4372/#review14300
---
Ship it!
Minor nit about the /* Safe */ comment.
- rmudgett
On Jan. 26, 2015, 6:14 p.m., rmudgett wrote:
/branches/13/apps/confbridge/include/confbridge.h, line 242
https://reviewboard.asterisk.org/r/4372/diff/1/?file=71081#file71081line242
This really should be a define instead of a magic number. Hmm... The
bridge profile name is also
On Jan. 26, 2015, 4:55 p.m., Scott Griepentrog wrote:
/branches/13/apps/confbridge/conf_config_parser.c, line 2304
https://reviewboard.asterisk.org/r/4372/diff/1/?file=71080#file71080line2304
Use ast_copy_string?
rmudgett wrote:
I second this should be ast_copy_string() for
On Jan. 26, 2015, 5:44 p.m., rmudgett wrote:
./branches/13/main/http.c, line 2155
https://reviewboard.asterisk.org/r/4374/diff/1/?file=71085#file71085line2155
Setting a blank string will really mean a blank servername output value:
Server:
Is it intended for this
Hi guys,
I noticed rtptimeout on asterisk 12 is not working, so i looked at the source.
looks like, it has no effect on res_rtp_asterisk?
Kelvin Chua
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
On Jan. 26, 2015, 5:44 p.m., rmudgett wrote:
./branches/13/main/http.c, lines 567-568
https://reviewboard.asterisk.org/r/4374/diff/1/?file=71085#file71085line567
Why are the allocation sizes passed in?
Ashley Sanders wrote:
To account for this handling the error and auth
---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4373/#review14291
---
Ship it!
Tested and the response now matches what is
On Jan. 26, 2015, 5:44 p.m., rmudgett wrote:
./branches/13/main/http.c, lines 608-615
https://reviewboard.asterisk.org/r/4374/diff/1/?file=71085#file71085line608
Yuck on toothbrush formatting.
I think this looks better, personally. It's easier to read and glean intent.
On Jan.
On Jan. 26, 2015, 5:44 p.m., rmudgett wrote:
./branches/13/main/http.c, line 2153
https://reviewboard.asterisk.org/r/4374/diff/1/?file=71085#file71085line2153
-1 is not needed for the sizeof parameter. The sizeof parameter
includes the string nul terminator.
I saw that
---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4377/#review14306
---
1) Server: Asterisk/version
2) Server: JohnMcClane
3)
---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4371/
---
Review request for Asterisk Developers.
Repository: Asterisk
---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4372/#review14290
---
/branches/13/apps/confbridge/conf_config_parser.c
---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4374/#review14289
---
./branches/13/configs/samples/http.conf.sample
---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4372/#review14292
---
/branches/13/apps/confbridge/include/confbridge.h
On Jan. 26, 2015, 4:55 p.m., Scott Griepentrog wrote:
/branches/13/apps/confbridge/conf_config_parser.c, line 2304
https://reviewboard.asterisk.org/r/4372/diff/1/?file=71080#file71080line2304
Use ast_copy_string?
I second this should be ast_copy_string() for safety.
- rmudgett
---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4351/#review14294
---
Ship it!
Ship It!
- rmudgett
On Jan. 17, 2015, 11:28 a.m.,
---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4374/#review14288
---
Ship it!
Ship It!
- Andrew Nagy
On Jan. 26, 2015, 12:03
43 matches
Mail list logo