apr-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ryan Bloom" <...@covalent.net>
Subject RE: cvs commit: apr-util/include apr_buckets.h
Date Sun, 02 Jun 2002 22:15:13 GMT
> From: Aaron Bannert [mailto:aaron@clove.org]
> 
> On Sun, Jun 02, 2002 at 12:42:30AM -0700, Ryan Bloom wrote:
> > > From: jwoolley@apache.org [mailto:jwoolley@apache.org]
> > >
> > >
> > >   -    int is_metadata;
> > >   +    enum {
> > >   +        /** This bucket type represents actual data to send to
the
> > > client. */
> > >   +        APR_BUCKET_DATA = 0,
> > >   +        /** This bucket type represents metadata. */
> > >   +        APR_BUCKET_METADATA = 1
> > >   +    } is_metadata;
> >
> > By turning this into an enum, you have removed the ability to extend
> > this field.
> 
> In what way?

This field can only have two values, DATA or METADATA.  It is unlikely
that we will add more, unless somebody tells us that they need it, but
it will take a release to get that change into their hands.  If this is
an integer, then anybody can extend it simply by choosing another int
value.

Ryan



Mime
View raw message