[FFmpeg-devel] [PATCH 1/8] Deprecate avcodec_get_pix_fmt_name() in favor of the new av_get_pix_fmt_name().

[Stefano Sabatini]

On date Monday 2010-11-08 13:23:53 +0100, Tomas H?rdin encoded:
> On Sun, 2010-11-07 at 15:06 +0100, Michael Niedermayer wrote:
> > On Sun, Nov 07, 2010 at 11:37:54AM +0100, Stefano Sabatini wrote:
> > > On date Sunday 2010-11-07 00:57:08 +0100, Michael Niedermayer encoded:
> > > > On Sat, Nov 06, 2010 at 02:12:43PM +0100, Stefano Sabatini wrote:
> > > > > On date Saturday 2010-11-06 03:07:13 +0100, Michael Niedermayer encoded:
> > > > > > On Fri, Nov 05, 2010 at 12:00:32PM +0100, Stefano Sabatini wrote:
> > > > > > > Consistent with av_get_sample_fmt_name().
> > > > > [...]
> > > > > > > --- a/libavutil/pixdesc.c
> > > > > > > +++ b/libavutil/pixdesc.c
> > > > > > > @@ -814,6 +814,14 @@ static enum PixelFormat get_pix_fmt_internal(const char *name)
> > > > > > >      return PIX_FMT_NONE;
> > > > > > >  }
> > > > > > >  
> > > > > > > +const char *av_get_pix_fmt_name(enum PixelFormat pix_fmt)
> > > > > > > +{
> > > > > > > +    if (pix_fmt < 0 || pix_fmt >= PIX_FMT_NB)
> > > > > > > +        return NULL;
> > > > > > > +    else
> > > > > > > +        return av_pix_fmt_descriptors[pix_fmt].name;
> > > > > > > +}
> > > > > > > +
> > > > > > 
> > > > > > iam against adding a function per field of a public structure.
> > > > > 
> > > > > Consistent with av_get_sample_fmt_name() so the user will expect to
> > > > 
> > > > The sample formats are desribed in a private struct, the pixel formats
> > > 
> > > I know all this, and I repeat what I already said,
> > > av_pix_fmt_descriptors[pix_fmt].name is an awkward way for getting the
> > > name of a pixel format and less safe (no check on pix_fmt range),
> > 
> > as said then add a function to get the descriptor
> > adding a function that gets 1 field from a descriptor is perverse
> > That field has no special meaning compared to other fields and our API is
> > complex enough as it is. And we surely dont want to have a function for each
> > field.
> 
> +1
> 
> This also allows Windows users to access those fields. See a thread a
> while back regarding that.

Yes and I remember it got stucked for some reason (and help in this
area is welcome, that is don't expect a patch from me).
 
> Also, aren't we doing hierarchical names now, so something like
> av_pix_fmt_get()?
-- 
FFmpeg = Faithful and Fascinating Moronic Power Erroneous Gargoyle