tools/venc_data_dump: factor out demux/decode code

It can be shared with other simple demux/decode tools.

tests/ref/fate/source

@@ -20,5 +20,6 @@ Headers without standard inclusion guards:
 compat/djgpp/math.h
 compat/float/float.h
 compat/float/limits.h
+tools/decode_simple.h
 Use of av_clip() where av_clip_uintp2() could be used:
 Use of av_clip() where av_clip_intp2() could be used:

tools/Makefile

@@ -17,6 +17,8 @@ tools/target_dem_fuzzer.o: tools/target_dem_fuzzer.c
 tools/target_io_dem_fuzzer.o: tools/target_dem_fuzzer.c
 	$(COMPILE_C) -DIO_FLAT=0
 
+tools/venc_data_dump$(EXESUF): tools/decode_simple.o
+
 OUTDIRS += tools
 
 clean::

tools/decode_simple.c

@@ -0,0 +1,157 @@
+/*
+ * This file is part of FFmpeg.
+ *
+ * FFmpeg is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Lesser General Public
+ * License as published by the Free Software Foundation; either
+ * version 2.1 of the License, or (at your option) any later version.
+ *
+ * FFmpeg is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ * Lesser General Public License for more details.
+ *
+ * You should have received a copy of the GNU Lesser General Public
+ * License along with FFmpeg; if not, write to the Free Software
+ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
+ */
+
+/* shared code for simple demux/decode tools */
+
+#include <stdlib.h>
+#include <string.h>
+
+#include "decode_simple.h"
+
+#include "libavformat/avformat.h"
+
+#include "libavcodec/avcodec.h"
+#include "libavcodec/packet.h"
+
+#include "libavutil/dict.h"
+#include "libavutil/error.h"
+#include "libavutil/frame.h"
+
+static int decode_read(DecodeContext *dc, int flush)
+{
+    const int ret_done = flush ? AVERROR_EOF : AVERROR(EAGAIN);
+    int ret = 0;
+
+    while (ret >= 0 &&
+           (dc->max_frames == 0 || dc->decoder->frame_number < dc->max_frames)) {
+        ret = avcodec_receive_frame(dc->decoder, dc->frame);
+        if (ret < 0) {
+            if (ret == AVERROR_EOF) {
+                int err = dc->process_frame(dc, NULL);
+                if (err < 0)
+                    return err;
+            }
+
+            return (ret == ret_done) ? 0 : ret;
+        }
+
+        ret = dc->process_frame(dc, dc->frame);
+        av_frame_unref(dc->frame);
+        if (ret < 0)
+            return ret;
+
+        if (dc->max_frames && dc->decoder->frame_number == dc->max_frames)
+            return 1;
+    }
+
+    return (dc->max_frames == 0 || dc->decoder->frame_number < dc->max_frames) ? 0 : 1;
+}
+
+int ds_run(DecodeContext *dc)
+{
+    int ret;
+
+    ret = avcodec_open2(dc->decoder, NULL, &dc->decoder_opts);
+    if (ret < 0)
+        return ret;
+
+    while (ret >= 0) {
+        ret = av_read_frame(dc->demuxer, dc->pkt);
+        if (ret < 0)
+            goto flush;
+        if (dc->pkt->stream_index != dc->stream->index) {
+            av_packet_unref(dc->pkt);
+            continue;
+        }
+
+        ret = avcodec_send_packet(dc->decoder, dc->pkt);
+        if (ret < 0) {
+            fprintf(stderr, "Error decoding: %d\n", ret);
+            return ret;
+        }
+        av_packet_unref(dc->pkt);
+
+        ret = decode_read(dc, 0);
+        if (ret < 0) {
+            fprintf(stderr, "Error decoding: %d\n", ret);
+            return ret;
+        } else if (ret > 0)
+            return 0;
+    }
+
+flush:
+    avcodec_send_packet(dc->decoder, NULL);
+    ret = decode_read(dc, 1);
+    if (ret < 0) {
+        fprintf(stderr, "Error flushing: %d\n", ret);
+        return ret;
+    }
+
+    return 0;
+}
+
+void ds_free(DecodeContext *dc)
+{
+    av_dict_free(&dc->decoder_opts);
+
+    av_frame_free(&dc->frame);
+    av_packet_free(&dc->pkt);
+
+    avcodec_free_context(&dc->decoder);
+    avformat_close_input(&dc->demuxer);
+}
+
+int ds_open(DecodeContext *dc, const char *url, int stream_idx)
+{
+    const AVCodec *codec;
+    int ret;
+
+    memset(dc, 0, sizeof(*dc));
+
+    dc->pkt   = av_packet_alloc();
+    dc->frame = av_frame_alloc();
+    if (!dc->pkt || !dc->frame) {
+        ret = AVERROR(ENOMEM);
+        goto fail;
+    }
+
+    ret = avformat_open_input(&dc->demuxer, url, NULL, NULL);
+    if (ret < 0) {
+        fprintf(stderr, "Error opening input file: %d\n", ret);
+        return ret;
+    }
+
+    if (stream_idx < 0 || stream_idx >= dc->demuxer->nb_streams)
+        return AVERROR(EINVAL);
+
+    dc->stream = dc->demuxer->streams[stream_idx];
+
+    codec = avcodec_find_decoder(dc->stream->codecpar->codec_id);
+    if (!codec)
+        return AVERROR_DECODER_NOT_FOUND;
+
+    dc->decoder = avcodec_alloc_context3(codec);
+    if (!dc->decoder)
+        return AVERROR(ENOMEM);
+
+    return 0;
+
+fail:
+    ds_free(dc);
+    return ret;
+}

tools/decode_simple.h

@@ -0,0 +1,53 @@
+/*
+ * This file is part of FFmpeg.
+ *
+ * FFmpeg is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Lesser General Public
+ * License as published by the Free Software Foundation; either
+ * version 2.1 of the License, or (at your option) any later version.
+ *
+ * FFmpeg is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ * Lesser General Public License for more details.
+ *
+ * You should have received a copy of the GNU Lesser General Public
+ * License along with FFmpeg; if not, write to the Free Software
+ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
+ */
+
+/* shared code for simple demux/decode tools */
+
+#ifndef DECODE_SIMPLE_H
+#define DECODE_SIMPLE_H
+
+#include "libavformat/avformat.h"
+
+#include "libavcodec/avcodec.h"
+#include "libavcodec/packet.h"
+
+#include "libavutil/dict.h"
+#include "libavutil/frame.h"
+
+
+typedef struct DecodeContext {
+    AVFormatContext *demuxer;
+    AVStream        *stream;
+    AVCodecContext  *decoder;
+
+    AVPacket        *pkt;
+    AVFrame         *frame;
+
+    int (*process_frame)(struct DecodeContext *dc, AVFrame *frame);
+    void            *opaque;
+
+    AVDictionary    *decoder_opts;
+    int              max_frames;
+} DecodeContext;
+
+int ds_open(DecodeContext *dc, const char *url, int stream_idx);
+void ds_free(DecodeContext *dc);
+
+int ds_run(DecodeContext *dc);
+
+#endif /* DECODE_SIMPLE_H */

tools/venc_data_dump.c

@@ -20,6 +20,8 @@
 #include <stdint.h>
 #include <stdlib.h>
 
+#include "decode_simple.h"
+
 #include "libavutil/common.h"
 #include "libavutil/dict.h"
 #include "libavutil/error.h"
@@ -29,20 +31,14 @@
 
 #include "libavcodec/avcodec.h"
 
-static int decode_read(AVCodecContext *decoder, AVFrame *frame, int flush, int max_frames)
+static int process_frame(DecodeContext *dc, AVFrame *frame)
 {
-    const int ret_done = flush ? AVERROR_EOF : AVERROR(EAGAIN);
-    int ret = 0;
-
-    while (ret >= 0 &&
-           (max_frames == 0 ||  decoder->frame_number < max_frames)) {
     AVFrameSideData *sd;
 
-        ret = avcodec_receive_frame(decoder, frame);
+    if (!frame)
-        if (ret < 0)
+        return 0;
-            return (ret == ret_done) ? 0 : ret;
 
-        fprintf(stdout, "frame %d\n", decoder->frame_number - 1);
+    fprintf(stdout, "frame %d\n", dc->decoder->frame_number - 1);
 
     sd = av_frame_get_side_data(frame, AV_FRAME_DATA_VIDEO_ENC_PARAMS);
     if (sd) {
@@ -67,47 +63,12 @@ static int decode_read(AVCodecContext *decoder, AVFrame *frame, int flush, int m
         }
     }
 
-        av_frame_unref(frame);
-
-        if (max_frames && decoder->frame_number == max_frames)
-            return 1;
-    }
-
-    return (max_frames == 0 || decoder->frame_number < max_frames) ? 0 : 1;
-}
-
-static int decoder_init(AVFormatContext *demuxer, int stream_idx,
-                        AVCodecContext **dec, AVDictionary **opts)
-{
-    const AVCodec *codec;
-    int ret;
-
-    if (stream_idx < 0 || stream_idx >= demuxer->nb_streams)
-        return AVERROR(EINVAL);
-
-    codec = avcodec_find_decoder(demuxer->streams[stream_idx]->codecpar->codec_id);
-    if (!codec)
-        return AVERROR_DECODER_NOT_FOUND;
-
-    *dec = avcodec_alloc_context3(codec);
-    if (!*dec)
-        return AVERROR(ENOMEM);
-
-    ret = avcodec_open2(*dec, NULL, opts);
-    if (ret < 0)
-        return ret;
-
     return 0;
 }
 
 int main(int argc, char **argv)
 {
-    AVFormatContext *demuxer = NULL;
+    DecodeContext dc;
-    AVCodecContext  *decoder = NULL;
-    AVDictionary       *opts = NULL;
-
-    AVPacket *pkt   = NULL;
-    AVFrame  *frame = NULL;
 
     unsigned int stream_idx, max_frames;
     const char *filename, *thread_type = NULL, *nb_threads = NULL;
@@ -126,70 +87,23 @@ int main(int argc, char **argv)
         thread_type = argv[5];
     }
 
-    ret  = av_dict_set(&opts, "threads",          nb_threads,    0);
+    ret = ds_open(&dc, filename, stream_idx);
-    ret |= av_dict_set(&opts, "thread_type",      thread_type,   0);
+    if (ret < 0)
-    ret |= av_dict_set(&opts, "export_side_data", "venc_params", 0);
-
-    ret = avformat_open_input(&demuxer, filename, NULL, NULL);
-    if (ret < 0) {
-        fprintf(stderr, "Error opening input file: %d\n", ret);
-        return ret;
-    }
-
-    ret = decoder_init(demuxer, stream_idx, &decoder, &opts);
-    if (ret < 0) {
-        fprintf(stderr, "Error initializing decoder\n");
         goto finish;
-    }
 
-    pkt   = av_packet_alloc();
+    dc.process_frame = process_frame;
-    frame = av_frame_alloc();
+    dc.max_frames    = max_frames;
-    if (!pkt || !frame) {
-        ret = AVERROR(ENOMEM);
-        goto finish;
-    }
 
-    while (ret >= 0) {
+    ret  = av_dict_set(&dc.decoder_opts, "threads",          nb_threads,    0);
-        ret = av_read_frame(demuxer, pkt);
+    ret |= av_dict_set(&dc.decoder_opts, "thread_type",      thread_type,   0);
-        if (ret < 0)
+    ret |= av_dict_set(&dc.decoder_opts, "export_side_data", "venc_params", 0);
-            goto flush;
-        if (pkt->stream_index != stream_idx) {
-            av_packet_unref(pkt);
-            continue;
-        }
 
-        ret = avcodec_send_packet(decoder, pkt);
+    if (ret < 0)
-        if (ret < 0) {
-            fprintf(stderr, "Error decoding: %d\n", ret);
-            goto finish;
-        }
-        av_packet_unref(pkt);
-
-        ret = decode_read(decoder, frame, 0, max_frames);
-        if (ret < 0) {
-            fprintf(stderr, "Error decoding: %d\n", ret);
-            goto finish;
-        } else if (ret > 0) {
-            ret = 0;
         goto finish;
-        }
-    }
 
-flush:
+    ret = ds_run(&dc);
-    avcodec_send_packet(decoder, NULL);
-    ret = decode_read(decoder, frame, 1, max_frames);
-    if (ret < 0) {
-        fprintf(stderr, "Error flushing: %d\n", ret);
-        goto finish;
-    }
-    ret = 0;
 
 finish:
-    av_dict_free(&opts);
+    ds_free(&dc);
-    av_packet_free(&pkt);
-    av_frame_free(&frame);
-    avcodec_free_context(&decoder);
-    avformat_close_input(&demuxer);
-
     return ret;
 }