[FFmpeg-devel] [RFC] Request for pixdesc API review

[Michael Niedermayer]

On Tue, Nov 17, 2009 at 04:24:16PM +0100, Stefano Sabatini wrote:
> On date Tuesday 2009-11-17 14:51:09 +0100, Michael Niedermayer encoded:
> > On Mon, Nov 16, 2009 at 12:43:02AM +0100, Stefano Sabatini wrote:
> > [...]
> > >  pixdesc.h |    4 ++++
> > >  1 file changed, 4 insertions(+)
> > > d60486d284ea2688ea83ef088ef8d858fbb1a367  clarify-log2-chroma.patch
> > > Index: ffmpeg/libavutil/pixdesc.h
> > > ===================================================================
> > > --- ffmpeg.orig/libavutil/pixdesc.h	2009-11-16 00:09:44.000000000 +0100
> > > +++ ffmpeg/libavutil/pixdesc.h	2009-11-16 00:11:10.000000000 +0100
> > > @@ -62,6 +62,8 @@
> > >       * For YV12 this is 1 for example.
> > >       * chroma_width = -((-luma_width) >> log2_chroma_w)
> > >       * The note above is needed to ensure rounding up.
> > > +     * This value only refers to the components 1 and 2 in the
> > > +     * descriptor.
> > >       */
> > >      uint8_t log2_chroma_w;      ///< chroma_width = -((-luma_width )>>log2_chroma_w)
> > 
> > it refers to chroma as its name says, i think thats clear already
> 
> But it's not immediately obvious that the chroma components are stored
> in comp[1] and comp[2].

well, then commit that change

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

No great genius has ever existed without some touch of madness. -- Aristotle
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20091117/8cf3e0a1/attachment.pgp>