[FFmpeg-devel] [PATCH 3/3] fate/oggvorbis: Fix tests after fixing AV_PKT_DATA_SKIP_SAMPLES

[Lynne]

7 Jul 2021, 17:41 by sunguangyucn at gmail.com:

> After fixing AV_PKT_DATA_SKIP_SAMPLES for reading vorbis packets from ogg,
> the actual decoded samples become fewer. Three fate tests are failing:
>
> fate-vorbis-20:
> The samples in 6.ogg are not frame aligned. 6.pcm file was generated by
> ffmpeg before the fix. After the fix, the decoded pcm file does not match
> anymore. The ref file 6.pcm needs to be updated.
>
> fate-webm-dash-chapters:
> The original vorbis_chapter_extension_demo.ogg is transmuxed to dash-webm.
> The ref file webm-dash-chapters needs to be updated.
>
> fate-vorbis-encode:
> This exposes another bug in the vorbis encoder that initial_padding is not
> correctly set. It is fixed in the previous patch.
>
> Signed-off-by: Guangyu Sun <gsun at roblox.com>
> ---
>  tests/fate/vorbis.mak             | 2 +-
>  tests/ref/fate/webm-dash-chapters | 4 ++--
>  2 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/tests/fate/vorbis.mak b/tests/fate/vorbis.mak
> index 354cc57a0f..65fcadf33d 100644
> --- a/tests/fate/vorbis.mak
> +++ b/tests/fate/vorbis.mak
> @@ -86,7 +86,7 @@ fate-vorbis-19: REF = $(SAMPLES)/vorbis/test-short2_small.pcm
>  
>  FATE_VORBIS += fate-vorbis-20
>  fate-vorbis-20: CMD = pcm -i $(TARGET_SAMPLES)/vorbis/6.ogg
> -fate-vorbis-20: REF = $(SAMPLES)/vorbis/6.pcm
> +fate-vorbis-20: REF = $(SAMPLES)/vorbis/6_v2.pcm 
>

I don't think it's really worth it including another copy of the same file,
only smaller. You can change the test itself to ignore the length difference
(but still keep the value check) by adding this line instead:
fate-vorbis-20: SIZE_TOLERANCE = XXXX

I'm not sure the exact units the tolerance is in, but try using multiples
or even divisors of 9948, +-1 until it fixes the check exactly.