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.
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to