diff options
| author | Martin Storsjö <martin@martin.st> | 2024-04-03 14:43:14 +0300 |
|---|---|---|
| committer | Martin Storsjö <martin@martin.st> | 2024-04-11 14:08:55 +0300 |
| commit | e4f5c2414bb50df07dce9a8860aaeeaccaed15cf (patch) | |
| tree | 9eb82324333e064ccfd6e496eb9b4e975e19520c | |
| parent | 497a0f839d7d98e182aa27044e0dbeeb89783d4b (diff) | |
| download | ffmpeg-e4f5c2414bb50df07dce9a8860aaeeaccaed15cf.tar.gz | |
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 f234e8a32e6c69d7b63f8627f278be7c2c987f43.
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>
| -rw-r--r-- | libavformat/tests/movenc.c | 10 | ||||
| -rw-r--r-- | tests/ref/fate/movenc | 2 |
2 files changed, 9 insertions, 3 deletions
diff --git a/libavformat/tests/movenc.c b/libavformat/tests/movenc.c index bb115b0821..2fd5c67e76 100644 --- a/libavformat/tests/movenc.c +++ b/libavformat/tests/movenc.c @@ -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). diff --git a/tests/ref/fate/movenc b/tests/ref/fate/movenc index f6a408aa54..5c12aeb29f 100644 --- a/tests/ref/fate/movenc +++ b/tests/ref/fate/movenc @@ -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 |