[asterisk-dev] [Code Review] Add variable length displayprompt packet to skinny and use octals

wedhorn reviewboard at asterisk.org
Fri Feb 1 00:45:31 CST 2013


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/2294/
-----------------------------------------------------------

(Updated Feb. 1, 2013, 12:45 a.m.)


Review request for Asterisk Developers and Brad Latus.


Changes
-------

Updated to use bit masking (as per opticron's comment to https://reviewboard.asterisk.org/r/2290/) and for changes already committed.


Summary
-------

Add new variable length displayprompt packet (0x0145) to skinny. This uses a similar method to the variable length callinfo packet in https://reviewboard.asterisk.org/r/2290/. Uses the new packet if the device is reporting protocol versions >= 17.

Add the use of octal codes for sending prompts to both the new and old displayprompt messages (also cleaned up soft_key_template_default to use the defined octal codes).


Diffs (updated)
-----

  /trunk/channels/chan_skinny.c 380755 

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


Testing
-------

Several calls on devices with protocol versions of 6 and 17. Prompts as expected.


Thanks,

wedhorn

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


More information about the asterisk-dev mailing list