[asterisk-dev] [Code Review] Add service message support for the 'national' switchtype

[rmudgett at digium.com]

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

Ship it!


Looks good other than the minor formatting comments noted.


/branches/1.4/q931.c
<https://reviewboard.asterisk.org/r/347/#comment2576>

    Excess spacing around ().



/branches/1.4/q931.c
<https://reviewboard.asterisk.org/r/347/#comment2577>

    Excess spacing around ().


- rmudgett


On 2009-09-01 14:25:57, Dwayne Hubbard wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/347/
> -----------------------------------------------------------
> 
> (Updated 2009-09-01 14:25:57)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Summary
> -------
> 
> Extended libpri service message support to correctly handle 'national' switchtype service messages.
> 
> Loosened chan_dahdi configuration restriction to allow the enabling of service_message_support when configured for the 'national' switchtype
> 
> development branches:
> https://origsvn.digium.com/svn/libpri/team/dhubbard/bbfcfm
> https://origsvn.digium.com/svn/asterisk/team/dhubbard/bbfcfm
> 
> 
> This addresses bug https://issues.asterisk.org/view.php?id=15803.
>     http://issues.asterisk.org/view.php?id=https://issues.asterisk.org/view.php?id=15803
> 
> 
> Diffs
> -----
> 
>   /branches/1.4/pri_q931.h 1058 
>   /branches/1.4/q931.c 1058 
> 
> Diff: https://reviewboard.asterisk.org/r/347/diff
> 
> 
> Testing
> -------
> 
> Michael Young (IRC nick: elguero) successfully tested on a live NI-2 switch
> 
> Tested Branch Revisions:
> https://origsvn.digium.com/svn/libpri/team/dhubbard/bbfcfm [^] r1039
> https://origsvn.digium.com/svn/asterisk/team/dhubbard/bbfcfm [^] r214818
> 
> Results:
> In the quote below, "maintenance mode == 'pri service disable'"
> 
> Quote Michael Young:
> 
> 1. I placed all the calls into maintenance mode. I tried calling in and
> received the message that the call could not go through.
> 
> 2. I placed one channel out of maintenance mode and tried again resulting in the call going through like it should. I tried another call (while keeping that channel up) and received the message again.
> 
> 3. I placed another channel out of maintenance mode and placed another call resulting in it going through as well.
> 
> 4. I then proceeded to put the rest of the channels out of maintenance mode and everything looked like it was back to normal. 
> 
> 
> Thanks,
> 
> Dwayne
> 
>