aboutsummaryrefslogtreecommitdiffstats
path: root/tools
diff options
context:
space:
mode:
authorAnton Khirnov <anton@khirnov.net>2022-10-18 13:37:25 +0200
committerAnton Khirnov <anton@khirnov.net>2023-03-28 13:00:42 +0200
commit992e984fa86ed4447082be4915996908b6acf3b5 (patch)
tree21bee22c3a3f5e2a0723f1235e42af62bed4788a /tools
parent9bf1848acfb7cbdb7f8dcaa07649467dfb3cec7b (diff)
downloadffmpeg-992e984fa86ed4447082be4915996908b6acf3b5.tar.gz
tools/decode_simple: always call process_frame(NULL) at the end
Currently this would not be done if max_frames is triggered. Makes no difference in either of the tools currently using decode_simple, but may be important in future tools.
Diffstat (limited to 'tools')
-rw-r--r--tools/decode_simple.c8
1 files changed, 4 insertions, 4 deletions
diff --git a/tools/decode_simple.c b/tools/decode_simple.c
index e02323064d..8a9e7c5f71 100644
--- a/tools/decode_simple.c
+++ b/tools/decode_simple.c
@@ -73,7 +73,7 @@ int ds_run(DecodeContext *dc)
while (ret >= 0) {
ret = av_read_frame(dc->demuxer, dc->pkt);
if (ret < 0)
- goto flush;
+ break;
if (dc->pkt->stream_index != dc->stream->index) {
av_packet_unref(dc->pkt);
continue;
@@ -91,10 +91,9 @@ int ds_run(DecodeContext *dc)
fprintf(stderr, "Error decoding: %d\n", ret);
return ret;
} else if (ret > 0)
- return 0;
+ goto finish;
}
-flush:
avcodec_send_packet(dc->decoder, NULL);
ret = decode_read(dc, 1);
if (ret < 0) {
@@ -102,7 +101,8 @@ flush:
return ret;
}
- return 0;
+finish:
+ return dc->process_frame(dc, NULL);
}
void ds_free(DecodeContext *dc)