[FFmpeg-devel] [PATCH] Implement av_find_pix_fmt_descriptor* functions

Stefano Sabatini stefano.sabatini-lala
Sun Mar 22 18:39:34 CET 2009


On date Sunday 2009-03-22 16:50:06 +0100, Michael Niedermayer encoded:
> On Sun, Mar 22, 2009 at 09:19:43AM +0100, Stefano Sabatini wrote:
> > On date Sunday 2009-03-22 02:13:58 +0100, Michael Niedermayer encoded:
> > > On Sun, Mar 22, 2009 at 01:47:17AM +0100, Stefano Sabatini wrote:
> > > > Hi all.
> > > > 
> > > > I'm not sure if is better:
> > > > av_find_pix_fmt_descriptor()
> > > > av_find_pix_fmt_descriptor_by_name()
> > > > 
> > > > which is similar to:
> > > > avcodec_find_decoder()
> > > > avcodec_find_decoder_by_name()
> > > > 
> > > > or
> > > > av_get_pix_fmt_descriptor()
> > > > av_get_pix_fmt_descriptor_by_name()
> > > > 
> > > > which is more similar to:
> > > > avcodec_get_pix_fmt()
> > > > 
> > > > I think "get" is more appropriate for the first case
> > > > (av_get_pix_fmt_descriptor) since we don't have to execute a search
> > > > but just pickup the pixdesc from the internal array, while "find"
> > > > looks more appropriate in the second case.
> > > > 
> > > > Anyway I don't have a strong opinion, so I'd like to hear your
> > > > opinion.
> > > > 
> > > > As for the little test I'll drop it when committing.
> > > 
> > > what is the point in wraper functions around
> > > array[x]
> > > ?
> > 
> > * the pix_fmt_desc array is not public
> 
> having the descriptors public but not the array makes no sense
> having neither public makes your function useless

I'm aware that pixdesc.h is not public, but I meant to export it to
the pixdesc.h header, like in attached patch.

Regards.
-- 
FFmpeg = Freak & Faboulous Majestic Pitiful Elaborated Glue
-------------- next part --------------
A non-text attachment was scrubbed...
Name: export-pix-desc.patch
Type: text/x-diff
Size: 1264 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090322/7fc88ce2/attachment.patch>



More information about the ffmpeg-devel mailing list