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

Tilghman Lesher tlesher at digium.com
Thu Oct 15 19:17:49 CDT 2009


On Thursday 15 October 2009 07:40:10 pm Kevin P. Fleming wrote:
> SVN commits to the Asterisk project wrote:
> > Author: tilghman
> > Date: Thu Oct 15 17:33:30 2009
> > New Revision: 224225
> >
<snip>
> > +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?

I was following the convention established by the other enum in the same
file (include/asterisk/app.h) for ast_ivr_option.  Is there an advantage to
the other enum, or is that a grandfathered exception?

-- 
Tilghman



More information about the asterisk-dev mailing list