[FFmpeg-devel] [PATCH 0/2] Fix VC-2 encoder

Christophe Gisquet christophe.gisquet at gmail.com
Tue May 3 19:06:21 CEST 2016


The encoder was leaving uninitialized data in the padding of slices,
while the specs seem to mandate the use of 0xFF. This is also the case
for the slice prefix, but it seems completely unused.

To validate this, classical vsynth encoding/decoding fate tests for
all supported chroma formats are added. Suggestions for being even
more concise in the target/rules are welcome.

Christophe Gisquet (2):
  vc2enc: prevent random data
  vc2: fate tests

 libavcodec/vc2enc.c                     |  4 ++++
 tests/fate/vcodec.mak                   | 17 ++++++++++++++++-
 tests/ref/vsynth/vsynth1-vc2-420p       |  4 ++++
 tests/ref/vsynth/vsynth1-vc2-420p10     |  4 ++++
 tests/ref/vsynth/vsynth1-vc2-420p12     |  4 ++++
 tests/ref/vsynth/vsynth1-vc2-422p       |  4 ++++
 tests/ref/vsynth/vsynth1-vc2-422p10     |  4 ++++
 tests/ref/vsynth/vsynth1-vc2-422p12     |  4 ++++
 tests/ref/vsynth/vsynth1-vc2-444p       |  4 ++++
 tests/ref/vsynth/vsynth1-vc2-444p10     |  4 ++++
 tests/ref/vsynth/vsynth1-vc2-444p12     |  4 ++++
 tests/ref/vsynth/vsynth2-vc2-420p       |  4 ++++
 tests/ref/vsynth/vsynth2-vc2-420p10     |  4 ++++
 tests/ref/vsynth/vsynth2-vc2-420p12     |  4 ++++
 tests/ref/vsynth/vsynth2-vc2-422p       |  4 ++++
 tests/ref/vsynth/vsynth2-vc2-422p10     |  4 ++++
 tests/ref/vsynth/vsynth2-vc2-422p12     |  4 ++++
 tests/ref/vsynth/vsynth2-vc2-444p       |  4 ++++
 tests/ref/vsynth/vsynth2-vc2-444p10     |  4 ++++
 tests/ref/vsynth/vsynth2-vc2-444p12     |  4 ++++
 tests/ref/vsynth/vsynth_lena-vc2-420p   |  4 ++++
 tests/ref/vsynth/vsynth_lena-vc2-420p10 |  4 ++++
 tests/ref/vsynth/vsynth_lena-vc2-420p12 |  4 ++++
 tests/ref/vsynth/vsynth_lena-vc2-422p   |  4 ++++
 tests/ref/vsynth/vsynth_lena-vc2-422p10 |  4 ++++
 tests/ref/vsynth/vsynth_lena-vc2-422p12 |  4 ++++
 tests/ref/vsynth/vsynth_lena-vc2-444p   |  4 ++++
 tests/ref/vsynth/vsynth_lena-vc2-444p10 |  4 ++++
 tests/ref/vsynth/vsynth_lena-vc2-444p12 |  4 ++++
 29 files changed, 128 insertions(+), 1 deletion(-)
 create mode 100644 tests/ref/vsynth/vsynth1-vc2-420p
 create mode 100644 tests/ref/vsynth/vsynth1-vc2-420p10
 create mode 100644 tests/ref/vsynth/vsynth1-vc2-420p12
 create mode 100644 tests/ref/vsynth/vsynth1-vc2-422p
 create mode 100644 tests/ref/vsynth/vsynth1-vc2-422p10
 create mode 100644 tests/ref/vsynth/vsynth1-vc2-422p12
 create mode 100644 tests/ref/vsynth/vsynth1-vc2-444p
 create mode 100644 tests/ref/vsynth/vsynth1-vc2-444p10
 create mode 100644 tests/ref/vsynth/vsynth1-vc2-444p12
 create mode 100644 tests/ref/vsynth/vsynth2-vc2-420p
 create mode 100644 tests/ref/vsynth/vsynth2-vc2-420p10
 create mode 100644 tests/ref/vsynth/vsynth2-vc2-420p12
 create mode 100644 tests/ref/vsynth/vsynth2-vc2-422p
 create mode 100644 tests/ref/vsynth/vsynth2-vc2-422p10
 create mode 100644 tests/ref/vsynth/vsynth2-vc2-422p12
 create mode 100644 tests/ref/vsynth/vsynth2-vc2-444p
 create mode 100644 tests/ref/vsynth/vsynth2-vc2-444p10
 create mode 100644 tests/ref/vsynth/vsynth2-vc2-444p12
 create mode 100644 tests/ref/vsynth/vsynth_lena-vc2-420p
 create mode 100644 tests/ref/vsynth/vsynth_lena-vc2-420p10
 create mode 100644 tests/ref/vsynth/vsynth_lena-vc2-420p12
 create mode 100644 tests/ref/vsynth/vsynth_lena-vc2-422p
 create mode 100644 tests/ref/vsynth/vsynth_lena-vc2-422p10
 create mode 100644 tests/ref/vsynth/vsynth_lena-vc2-422p12
 create mode 100644 tests/ref/vsynth/vsynth_lena-vc2-444p
 create mode 100644 tests/ref/vsynth/vsynth_lena-vc2-444p10
 create mode 100644 tests/ref/vsynth/vsynth_lena-vc2-444p12

-- 
2.8.1



More information about the ffmpeg-devel mailing list