> From: Aaron Bannert [mailto:[EMAIL PROTECTED]
> 
> On Sun, Jun 02, 2002 at 12:42:30AM -0700, Ryan Bloom wrote:
> > > From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED]
> > >
> > >
> > >   -    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


Reply via email to