Hi, 2011/7/12 Måns Rullgård <[email protected]>: > "Ronald S. Bultje" <[email protected]> writes: >> On Tue, Jul 12, 2011 at 2:02 AM, Kostya <[email protected]> wrote: >>> On Tue, Jul 12, 2011 at 10:52:19AM +0200, Oskar Arvidsson wrote: >>>> --- >>>> libavutil/pixdesc.c | 6 +++--- >>>> 1 files changed, 3 insertions(+), 3 deletions(-) >>>> >>>> diff --git a/libavutil/pixdesc.c b/libavutil/pixdesc.c >>>> index efc7c7e..c70a413 100644 >>>> --- a/libavutil/pixdesc.c >>>> +++ b/libavutil/pixdesc.c >>>> @@ -918,9 +918,9 @@ const AVPixFmtDescriptor >>>> av_pix_fmt_descriptors[PIX_FMT_NB] = { >>>> .log2_chroma_w= 0, >>>> .log2_chroma_h= 0, >>>> .comp = { >>>> - {0,1,1,0,9}, /* Y */ >>>> - {1,1,1,0,9}, /* U */ >>>> - {2,1,1,0,9}, /* V */ >>>> + {0,1,1,0,8}, /* Y */ >>>> + {1,1,1,0,8}, /* U */ >>>> + {2,1,1,0,8}, /* V */ >>>> }, >>>> .flags = PIX_FMT_BE, >>>> }, >>> >>> looks correct >> >> *shame* - indeed. Queued. > > Is this not covered by fate, or did someone forget to update the checksums?
I don't think the value is used in practice. It could be used for clipping or shifting or so, but most of that code directly hardcodes values based on PIX_FMT_* values. Ronald _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
