[asterisk-dev] [Code Review] Use the proper type for ast_codec_pref's framing field so that packetization values > 127 can be used

Terry Wilson reviewboard at asterisk.org
Wed Dec 21 23:35:13 CST 2011

This is an automatically generated e-mail. To reply, visit:

Review request for Asterisk Developers.


ast_codec_pref has a framing field that is set as a signed char array even though every place in the code treats it as an int array. This just changes the field to an int array. It uses a magic number, but it is essentially the same thing that was there before. We could probably move the AST_FORMAT_LIST into the header and use its ARRAY_LEN as the size so that things are no bigger than they need to be. I figured I'd go with the smallest change first and see what people thought.

This addresses bug ASTERISK-18876.


  /branches/1.8/include/asterisk/frame.h 348793 

Diff: https://reviewboard.asterisk.org/r/1639/diff


Made calls with allow=g729:230 and saw that it was used whereas before it looped around to a lower value.



-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20111222/cac0bf19/attachment.htm>

More information about the asterisk-dev mailing list