On Fri, Jul 27, 2012 at 07:19:49PM +0600, Mashiat Sarker Shakkhar wrote:
> On 7/27/2012 7:04 PM, Kostya Shishkov wrote:
> >On Fri, Jul 27, 2012 at 06:49:44PM +0600, Mashiat Sarker Shakkhar wrote:
> >>From: Michael Niedermayer <[email protected]>
> >>
> >>Fixes read of uninitialized memory
> >>
> >>Signed-off-by: Michael Niedermayer <[email protected]>
> >>---
> >>  libavcodec/vc1dec.c        |    2 +-
> >>  tests/ref/fate/vc1_sa10143 |   58 
> >> ++++++++++++++++++++++----------------------
> >>  2 files changed, 30 insertions(+), 30 deletions(-)
> >
> >Are you sure it's proper way to go or is it just a hack to get rid of some
> >warning?
> 
> I am not really sure. For interlaced field P blocks, the filtering
> should be same as progressive - so this patch does not apply. On the
> other hand, for interlaced frame blocks, the progressive filtering
> is probably not right (we need to take fieldtx into account).
> 
> I would say, stick to what we know is right - hence filtering only
> progressive blocks.

May the section 10.10 be with you.
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to