[asterisk-dev] [Code Review] Update timing to deal with multiple ranges and per-minute accuracy

[Mark Michelson]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.digium.com/r/53/#review138
-----------------------------------------------------------



trunk/main/pbx.c
<http://reviewboard.digium.com/r/53/#comment248>

    I also just noticed that the indentation level of this function could be improved. If you add a return right here, then the corresponding else block can be shifted to the left by one level.


- Mark


On 2008-11-12 17:03:40, Tilghman Lesher wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.digium.com/r/53/
> -----------------------------------------------------------
> 
> (Updated 2008-11-12 17:03:40)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Summary
> -------
> 
> The timing API was mis-documented in TFOT to allow multiple ranges and single hours, through the use of the '&'.  This patch updates the code to make TFOT correct.  Also, it sets times to be accurate down to the minute, instead of the current 2-minute precision that confuses so many people.
> 
> 
> Diffs
> -----
> 
>   trunk/include/asterisk/pbx.h 156387 
>   trunk/main/pbx.c 156387 
> 
> Diff: http://reviewboard.digium.com/r/53/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Tilghman
> 
>