[asterisk-dev] [Code Review] Add app_v110 to accept v.110 data calls

[Mark Michelson]

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



/trunk/apps/app_v110.c
<https://reviewboard.asterisk.org/r/1394/#comment8221>

    Actually, this is allowed by the coding guidelines, since the declaration is at the beginning of the for loop's scope.
    
    The guideline you are thinking of states that variables shouldn't be declared "mid-block." In other words, no mixed declarations and code. Since this declaration is at the top of the scope, it's fine.



/trunk/apps/app_v110.c
<https://reviewboard.asterisk.org/r/1394/#comment8222>

    Same comment here as well.


- Mark


On Aug. 26, 2011, 7:19 p.m., dwmw2 wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/1394/
> -----------------------------------------------------------
> 
> (Updated Aug. 26, 2011, 7:19 p.m.)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Summary
> -------
> 
> This adds a new application app_v110 which accepts data calls from a GSM mobile phone.
> 
> 
> This addresses bug ASTERISK-14185.
>     https://issues.asterisk.org/jira/browse/ASTERISK-14185
> 
> 
> Diffs
> -----
> 
>   /trunk/channels/chan_dahdi.c 333428 
>   /trunk/apps/app_v110.c PRE-CREATION 
> 
> Diff: https://reviewboard.asterisk.org/r/1394/diff
> 
> 
> Testing
> -------
> 
> Asterisk 1.8.5.0+app_v110, ISDN BRI on British Telecom, Nokia N97 GSM phone...
> 
> AT+CBST=71,0,1
> OK
> ATD01799xxxxxx
> CONNECT
> Fedora release 15 (Lovelock)
> Kernel 2.6.40.3-0.fc15.i686 on an i686 (/dev/pts/34)
> 
> obelisk.infradead.org login: dwmw2
> Password: 
> Last login: Sat Aug 27 01:02:52 2011 from 07976xxxxxx
> [dwmw2 at obelisk ~]$ 
> 
> 
> Thanks,
> 
> dwmw2
> 
>

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