[asterisk-dev] [Code Review] GSOC 2010 - Data API providers

[Tilghman Lesher]

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



/trunk/apps/app_meetme.c
<https://reviewboard.asterisk.org/r/757/#comment5035>

    Let's think about possibly creating a type that more specifically denotes this as a timestamp, perhaps AST_DATA_TIMESTAMP or AST_DATA_INTEGER_TIMESTAMP.  One possible application of this would be to be able to receive data both as the literal epoch timestamp, as well as a datetime formatted string, and for Asterisk to be able to do the right thing.



/trunk/apps/app_meetme.c
<https://reviewboard.asterisk.org/r/757/#comment5036>

    Could we create a type to denote that these values are in seconds or milliseconds, whatever the correct unit is?



/trunk/apps/app_voicemail.c
<https://reviewboard.asterisk.org/r/757/#comment5037>

    Probably should be ast_debug()



/trunk/channels/chan_dahdi.c
<https://reviewboard.asterisk.org/r/757/#comment5038>

    Seems like these should all be booleans, instead of an encoded string.



/trunk/channels/chan_dahdi.c
<https://reviewboard.asterisk.org/r/757/#comment5039>

    The ordering of these looks backwards.  According to this, if CRC4 is off, then the line is always "YEL", which should not be the case.


- Tilghman


On 2010-06-29 22:27:37, Eliel Sardañons wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/757/
> -----------------------------------------------------------
> 
> (Updated 2010-06-29 22:27:37)
> 
> 
> Review request for Asterisk Developers and Tilghman Lesher.
> 
> 
> Summary
> -------
> 
> Data API providers already implemented for the summer of code 2010 project
> 
> 
> Diffs
> -----
> 
>   /trunk/apps/app_meetme.c 273179 
>   /trunk/apps/app_queue.c 273179 
>   /trunk/apps/app_voicemail.c 273179 
>   /trunk/channels/chan_agent.c 273179 
>   /trunk/channels/chan_dahdi.c 273179 
>   /trunk/channels/chan_iax2.c 273179 
>   /trunk/channels/chan_sip.c 273179 
>   /trunk/include/asterisk/cdr.h 273179 
>   /trunk/include/asterisk/channel.h 273179 
>   /trunk/include/asterisk/data.h 273179 
>   /trunk/include/asterisk/indications.h 273179 
>   /trunk/main/cdr.c 273179 
>   /trunk/main/channel.c 273179 
>   /trunk/main/data.c 273179 
>   /trunk/main/indications.c 273179 
>   /trunk/res/res_odbc.c 273179 
> 
> Diff: https://reviewboard.asterisk.org/r/757/diff
> 
> 
> Testing
> -------
> 
> Developer testing.
> 
> 
> Thanks,
> 
> Eliel
> 
>