[FFmpeg-devel] [PATCH 03/17] fate/mxf: Add tests for H.264 remuxing

[Tomas Härdin]

tis 2021-11-09 klockan 19:01 +0100 skrev Andreas Rheinhardt:
> These tests exhibit two bugs: Instead of using the in-band extradata
> the demuxer makes up some extradata designed for AVC intra tracks
> that lack in-band extradata; these files are nevertheless decodable
> because of the in-band extradata. Furthermore, the frame reordering
> is lost.
> 
> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at outlook.com>
> ---
>  tests/fate/mxf.mak            | 22 ++++++++++-
>  tests/ref/fate/mxf-remux-h264 | 37 ++++++++++++++++++
>  tests/ref/fate/mxf-remux-xavc | 71
> +++++++++++++++++++++++++++++++++++
>  3 files changed, 128 insertions(+), 2 deletions(-)
>  create mode 100644 tests/ref/fate/mxf-remux-h264
>  create mode 100644 tests/ref/fate/mxf-remux-xavc
> 
> diff --git a/tests/fate/mxf.mak b/tests/fate/mxf.mak
> index f96f4a429b..58a697cd86 100644
> --- a/tests/fate/mxf.mak
> +++ b/tests/fate/mxf.mak
> @@ -42,6 +42,21 @@ FATE_MXF_REMUX_PROBE-$(call ALLYES, PRORES_DECODER
> MXF_MUXER) \
>                                      += fate-mxf-remux-applehdr10
>  fate-mxf-remux-applehdr10: CMD = transcode mxf
> $(TARGET_SAMPLES)/mxf/Meridian-Apple_ProResProxy-HDR10.mxf mxf "-map
> 0 -c copy" "-c copy -t 0.3" "" "-show_entries
> format_tags:stream_side_data_list:stream=index,codec_name,codec_tag:s
> tream_tags"
>  
> +# Tests muxing H.264, in particular automatic insertion of
> h264_mp4toannexb.
> +# FIXME: The timestamps of the demuxed file are not properly
> reordered.
> +# Furthermore the extradata is wrong: It is one of the AVC intra
> SPS/PPS;
> +# decoding only works due to in-band extradata.

Is this a problem with the samples or the code? Or both?

/Tomas