[asterisk-dev] [asterisk-commits] tilghman: trunk r224225 - in /trunk: include/asterisk/ main/

[Kevin P. Fleming]

SVN commits to the Asterisk project wrote:
> Author: tilghman
> Date: Thu Oct 15 17:33:30 2009
> New Revision: 224225
> 
> URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=224225
> Log:
> Create an API for adding an optional time unit onto the ends of time periods.
> Two examples of its use are included, and the usage could be expanded in some
> cases into certain configuration options where time periods are specified.
> 
> Modified:
>     trunk/include/asterisk/app.h
>     trunk/main/app.c
>     trunk/main/pbx.c
> 
> Modified: trunk/include/asterisk/app.h
> URL: http://svnview.digium.com/svn/asterisk/trunk/include/asterisk/app.h?view=diff&rev=224225&r1=224224&r2=224225
> ==============================================================================
> --- trunk/include/asterisk/app.h (original)
> +++ trunk/include/asterisk/app.h Thu Oct 15 17:33:30 2009
> @@ -84,6 +84,12 @@
>  	static struct ast_ivr_option __options_##holder[] = foo;\
>  	static struct ast_ivr_menu holder = { title, flags, __options_##holder }
>  
> +typedef enum {
> +	TIMELEN_HOURS,
> +	TIMELEN_MINUTES,
> +	TIMELEN_SECONDS,
> +	TIMELEN_MILLISECONDS,
> +} ast_timelen;

We don't use typedefs for enums or structs unless there is a specific
advantage to doing so; is there one here?

-- 
Kevin P. Fleming
Digium, Inc. | Director of Software Technologies
445 Jan Davis Drive NW - Huntsville, AL 35806 - USA
skype: kpfleming | jabber: kpfleming at digium.com
Check us out at www.digium.com & www.asterisk.org