[FFmpeg-cvslog] r25688 - trunk/libavcodec/imgconvert.c

[Michael Niedermayer]

On Sun, Nov 07, 2010 at 02:45:39PM +0100, Stefano Sabatini wrote:
> On date Sunday 2010-11-07 14:30:39 +0100, stefano wrote:
> > Author: stefano
> > Date: Sun Nov  7 14:30:39 2010
> > New Revision: 25688
> > 
> > Log:
> > Use av_pix_fmt_descriptors information in avcodec_pix_fmt_string().
> > 
> > Modified:
> >    trunk/libavcodec/imgconvert.c
> > 
> > Modified: trunk/libavcodec/imgconvert.c
> > ==============================================================================
> > --- trunk/libavcodec/imgconvert.c	Sun Nov  7 14:30:36 2010	(r25687)
> > +++ trunk/libavcodec/imgconvert.c	Sun Nov  7 14:30:39 2010	(r25688)
> > @@ -446,7 +446,7 @@ void avcodec_pix_fmt_string (char *buf, 
> >                    "%-11s %5d %9d",
> >                    av_pix_fmt_descriptors[pix_fmt].name,
> >                    info.nb_channels,
> > -                  info.depth
> > +                  av_get_bits_per_pixel(&av_pix_fmt_descriptors[pix_fmt])
> >              );
> 
> And now I see that this is wrong, as "depth" is not the same as the
> number of bits per pixel.
> 
> I suppose the better way to fix this is to change the header, and
> print "nb_bits" rather than "depth".

what is the diff this patch does to the output?

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

It is not what we do, but why we do it that matters.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-cvslog/attachments/20101107/0fca70d6/attachment.pgp>