RepoMirrors
/
ffmpeg
mirror of https://git.ffmpeg.org/ffmpeg.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

tests/movenc: Validate that normal muxer usage doesn't print warnings 

We have test to make sure that certain configurations do print
warnings. However, the normal operation of the muxer within this
test always printed a warning, so those tests to check for
extra warnings didn't essentially guard anything.

The warning that always was printed, "track 1: codec frame size is
not set" was not present in the libav fork where this testcase
originated, it was removed in f234e8a32e.

Set the frame size for the audio stream to silence the warning,
and use this frame size in a couple later calculations, and check
that one test configuration doesn't print warnings.

Setting the frame size apparently changes the rounding of a timestamp
in the ismv muxing testcase.

Signed-off-by: Martin Storsjö <martin@martin.st>
This commit is contained in:
Martin Storsjö 2024-04-03 14:43:14 +03:00
parent 497a0f839d
commit e4f5c2414b
2 changed files with 9 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
libavformat/tests/movenc.c
View File

@ -215,6 +215,7 @@ static void init_fps(int bf, int audio_preroll, int fps, int id3)
st->codecpar->codec_type = AVMEDIA_TYPE_AUDIO;
st->codecpar->codec_id = AV_CODEC_ID_AAC;
st->codecpar->sample_rate = 44100;
st->codecpar->frame_size = 1024;
st->codecpar->ch_layout = (AVChannelLayout)AV_CHANNEL_LAYOUT_STEREO;
st->time_base.num = 1;
st->time_base.den = 44100;
@ -243,9 +244,10 @@ static void init_fps(int bf, int audio_preroll, int fps, int id3)
frames = 0;
gop_size = 30;
duration = video_st->time_base.den / fps;
audio_duration = 1024LL * audio_st->time_base.den / audio_st->codecpar->sample_rate;
audio_duration = (long long)audio_st->codecpar->frame_size *
audio_st->time_base.den / audio_st->codecpar->sample_rate;
if (audio_preroll)
audio_preroll = 2048LL * audio_st->time_base.den / audio_st->codecpar->sample_rate;
audio_preroll = 2 * audio_duration;
bframes = bf;
video_dts = bframes ? -duration : 0;
@ -470,6 +472,7 @@ int main(int argc, char **argv)
// Similar to the previous one, but with input that doesn't start at
// pts/dts 0. avoid_negative_ts behaves in the same way as
// in non-empty-moov-no-elst above.
init_count_warnings();
init_out("empty-moov-no-elst");
av_dict_set(&opts, "movflags", "+frag_keyframe+empty_moov", 0);
init(1, 0);
@ -477,6 +480,9 @@ int main(int argc, char **argv)
finish();
close_out();
reset_count_warnings();
check(num_warnings == 0, "Unexpected warnings printed");
// Same as the previous one, but disable avoid_negative_ts (which
// would require using an edit list, but with empty_moov, one can't
// write a sensible edit list, when the start timestamps aren't known).

2
tests/ref/fate/movenc
View File

@ -20,7 +20,7 @@ write_data len 828, time nopts, type unknown atom -
write_data len 728, time 999999, type sync atom moof
write_data len 812, time nopts, type unknown atom -
write_data len 148, time nopts, type trailer atom -
92ce825ff40505ec8676191705adb7e7 4439 ismv
d2df24d323f4a8896441cd91203ac5f8 4439 ismv
write_data len 36, time nopts, type header atom ftyp
write_data len 1123, time nopts, type header atom -
write_data len 796, time 0, type sync atom moof