[asterisk-bugs] [JIRA] (ASTERISK-23592) [patch]Improvements to chan_unistim

[Rusty Newton (JIRA)]

    [ https://issues.asterisk.org/jira/browse/ASTERISK-23592?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=217171#comment-217171 ] 

Rusty Newton commented on ASTERISK-23592:
-----------------------------------------

Igor will likely be the primary person reviewing your code, however if you like, you can also use reviewboard.asterisk.org to facilitate the review process. Our overall code review process has recently been updated and is described on the [wiki at this link|https://wiki.asterisk.org/wiki/display/AST/Code+Review]. We still like to have a JIRA issue associated to the submission, but the reviewboard tool is designed for reviews. You can reference the reviewboard issue within the JIRA issue by editing this issue and finding the appropriate field.

Using reviewboard, may even make it easier for Igor to provide targeted feedback on sections of your code.

Thanks!

> [patch]Improvements to chan_unistim
> -----------------------------------
>
>                 Key: ASTERISK-23592
>                 URL: https://issues.asterisk.org/jira/browse/ASTERISK-23592
>             Project: Asterisk
>          Issue Type: Improvement
>      Security Level: None
>          Components: Channels/chan_unistim
>    Affects Versions: SVN
>            Reporter: Peter Whisker
>            Assignee: Igor Goncharovsky
>            Severity: Minor
>         Attachments: chan_unistim.c.diff, chan_unistim.c.diff
>
>
> I have been actively testing Unistim devices (i2002 & i2004) and have encountered issues with
> 1) dialtone wrong
> 2) inability to select DTMF playback duration
> 3) inability to set date to match than of a Nortel CS1000
> 4) mute causing playback to be muted as well as microphone
> 5) Timer displayed in French (Duree)
> The attached patch fixes as follows:
> 1) The "modulation" should not be referenced for tone+tone as it refers to the on-off characteristic I believe - this often resulted in a single tone rather than the multitone as in the UK.
> 2) I have added the unistim.conf variable dtmf_duration which can select the DTMF playback duration from 0ms to 150ms (0 is off and is the new default)
> 3) I have enabled the transmission of MonthLabels (in English) which are sent with the date and changed the dateformat variable to accept the values 0-3 as per the UNISTIM standard (2 & 3 match the previous 1 & 2 formats).
> 4) I have enabled the "Mute" packet and merged it into the previous code. It now only mutes the microphone. (Improvements welcome)
> 5) Changed Duree to Timer on i2004 display



--
This message was sent by Atlassian JIRA
(v6.2#6252)