Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
 help / color / mirror / Atom feed
From: Diederick Niehorster <dcnieho@gmail.com>
To: ffmpeg-devel@ffmpeg.org
Cc: Diederick Niehorster <dcnieho@gmail.com>
Subject: [FFmpeg-devel] [PATCH v4 04/22] avdevice/dshow: implement control_message interface
Date: Fri, 25 Mar 2022 15:10:23 +0100
Message-ID: <20220325141041.1748-5-dcnieho@gmail.com> (raw)
In-Reply-To: <20220325141041.1748-1-dcnieho@gmail.com>

This allows programmatic users of avdevice to start and stop the
DirectShow Capture graph (i.e. receive frames or not). This is important
because now the buffer fills up and starts dropping samples when
enqueued packets are not read out immediately after the demuxer is
opened.

Bumping avdevice version.

Signed-off-by: Diederick Niehorster <dcnieho@gmail.com>
---
 libavdevice/dshow.c         | 42 +++++++++++++++++++++++++++++++++++++
 libavdevice/dshow_capture.h |  1 +
 libavdevice/version.h       |  2 +-
 3 files changed, 44 insertions(+), 1 deletion(-)

diff --git a/libavdevice/dshow.c b/libavdevice/dshow.c
index abb8325bc3..652e093204 100644
--- a/libavdevice/dshow.c
+++ b/libavdevice/dshow.c
@@ -1503,6 +1503,45 @@ error:
     return ret;
 }
 
+static int dshow_control_message(AVFormatContext *avctx, int type, void *data, size_t data_size)
+{
+    struct dshow_ctx *ctx = avctx->priv_data;
+    int run_state = ctx->is_running;
+    HRESULT hr;
+
+    switch (type) {
+    case AV_APP_TO_DEV_PAUSE:
+        run_state = 0;
+        break;
+    case AV_APP_TO_DEV_PLAY:
+        run_state = 1;
+        break;
+    case AV_APP_TO_DEV_TOGGLE_PAUSE:
+        run_state = !run_state;
+        break;
+    }
+
+    // if play state change requested, apply
+    if (run_state != ctx->is_running) {
+        if (run_state)
+            hr = IMediaControl_Run(ctx->control);
+        else
+            hr = IMediaControl_Pause(ctx->control);
+
+        if (hr == S_FALSE) {
+            OAFilterState pfs;
+            hr = IMediaControl_GetState(ctx->control, 0, &pfs);
+        }
+        if (hr != S_OK) {
+            av_log(avctx, AV_LOG_ERROR, "Could not run/pause graph\n");
+            return AVERROR(EIO);
+        }
+        ctx->is_running = run_state;
+    }
+
+    return 0;
+}
+
 static enum AVCodecID waveform_codec_id(enum AVSampleFormat sample_fmt)
 {
     switch (sample_fmt) {
@@ -1747,6 +1786,7 @@ static int dshow_read_header(AVFormatContext *avctx)
         }
         // don't exit yet, allow it to list crossbar options in dshow_open_device
     }
+    ctx->is_running = 0;
     if (ctx->device_name[VideoDevice]) {
         if ((r = dshow_open_device(avctx, devenum, VideoDevice, VideoSourceDevice)) < 0 ||
             (r = dshow_add_device(avctx, VideoDevice)) < 0) {
@@ -1820,6 +1860,7 @@ static int dshow_read_header(AVFormatContext *avctx)
         av_log(avctx, AV_LOG_ERROR, "Could not run graph (sometimes caused by a device already in use by other application)\n");
         goto error;
     }
+    ctx->is_running = 1;
 
     ret = 0;
 
@@ -1932,6 +1973,7 @@ const AVInputFormat ff_dshow_demuxer = {
     .read_packet    = dshow_read_packet,
     .read_close     = dshow_read_close,
     .get_device_list= dshow_get_device_list,
+    .control_message= dshow_control_message,
     .flags          = AVFMT_NOFILE | AVFMT_NOBINSEARCH | AVFMT_NOGENSEARCH | AVFMT_NO_BYTE_SEEK,
     .priv_class     = &dshow_class,
 };
diff --git a/libavdevice/dshow_capture.h b/libavdevice/dshow_capture.h
index b548cd7afc..d0dd35a670 100644
--- a/libavdevice/dshow_capture.h
+++ b/libavdevice/dshow_capture.h
@@ -331,6 +331,7 @@ struct dshow_ctx {
 
     IMediaControl *control;
     IMediaEvent *media_event;
+    int is_running;
 
     enum AVPixelFormat pixel_format;
     enum AVCodecID video_codec_id;
diff --git a/libavdevice/version.h b/libavdevice/version.h
index 01e566a1be..69317c9280 100644
--- a/libavdevice/version.h
+++ b/libavdevice/version.h
@@ -31,7 +31,7 @@
 #include "version_major.h"
 
 #define LIBAVDEVICE_VERSION_MINOR   7
-#define LIBAVDEVICE_VERSION_MICRO 100
+#define LIBAVDEVICE_VERSION_MICRO 101
 
 #define LIBAVDEVICE_VERSION_INT AV_VERSION_INT(LIBAVDEVICE_VERSION_MAJOR, \
                                                LIBAVDEVICE_VERSION_MINOR, \
-- 
2.28.0.windows.1

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

  parent reply	other threads:[~2022-03-25 14:11 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-03-25 14:10 [FFmpeg-devel] [PATCH v4 00/22] avdevice (mostly dshow) enhancements Diederick Niehorster
2022-03-25 14:10 ` [FFmpeg-devel] [PATCH v4 01/22] avdevice/dshow: fix regression Diederick Niehorster
2022-03-25 14:10 ` [FFmpeg-devel] [PATCH v4 02/22] avdevice: lock to minor version of avformat Diederick Niehorster
2022-03-25 14:10 ` [FFmpeg-devel] [PATCH v4 03/22] avformat: add control_message function to AVInputFormat Diederick Niehorster
2022-03-25 14:10 ` Diederick Niehorster [this message]
2022-03-25 14:10 ` [FFmpeg-devel] [PATCH v4 05/22] avdevice: add control message requesting to show config dialog Diederick Niehorster
2022-03-25 14:10 ` [FFmpeg-devel] [PATCH v4 06/22] avdevice/dshow: accept show config dialog control message Diederick Niehorster
2022-03-25 14:10 ` [FFmpeg-devel] [PATCH v4 07/22] avdevice/dshow: add config dialog command for crossbar and tv tuner Diederick Niehorster
2022-03-25 14:10 ` [FFmpeg-devel] [PATCH v4 08/22] avdevice/avdevice: Revert "Deprecate AVDevice Capabilities API" Diederick Niehorster
2022-03-25 14:10 ` [FFmpeg-devel] [PATCH v4 09/22] avdevice/avdevice: clean up avdevice_capabilities_create Diederick Niehorster
2022-03-25 14:10 ` [FFmpeg-devel] [PATCH v4 10/22] avdevice: capabilities API details no longer public Diederick Niehorster
2022-03-25 14:10 ` [FFmpeg-devel] [PATCH v4 11/22] avutil/opt: document AVOptionRange min_value > max_value Diederick Niehorster
2022-03-25 14:10 ` [FFmpeg-devel] [PATCH v4 12/22] avdevice: Add internal helpers for querying device capabilities Diederick Niehorster
2022-03-25 14:10 ` [FFmpeg-devel] [PATCH v4 13/22] avdevice: change device capabilities option type Diederick Niehorster
2022-03-25 14:10 ` [FFmpeg-devel] [PATCH v4 14/22] avdevice: improve capabilities' option API Diederick Niehorster
2022-03-25 14:10 ` [FFmpeg-devel] [PATCH v4 15/22] avdevice/dshow: move audio format helpers Diederick Niehorster
2022-03-25 14:10 ` [FFmpeg-devel] [PATCH v4 16/22] avdevice/dshow: when closing, set context fields back to zero Diederick Niehorster
2022-03-25 14:10 ` [FFmpeg-devel] [PATCH v4 17/22] avdevice/dshow: implement capabilities API Diederick Niehorster
2022-03-25 14:10 ` [FFmpeg-devel] [PATCH v4 18/22] avdevice/dshow: cosmetics Diederick Niehorster
2022-03-25 14:10 ` [FFmpeg-devel] [PATCH v4 19/22] avformat: add avformat_alloc_input_context() Diederick Niehorster
2022-03-25 14:10 ` [FFmpeg-devel] [PATCH v4 20/22] doc/examples: adding device_get_capabilities example Diederick Niehorster
2022-03-25 17:26   ` Michael Niedermayer
2022-03-25 21:19     ` Diederick C. Niehorster
2022-03-25 14:10 ` [FFmpeg-devel] [PATCH v4 21/22] Makefile/examples: cosmetics Diederick Niehorster
2022-03-25 14:10 ` [FFmpeg-devel] [PATCH v4 22/22] avdevice/dshow: capabilities query also works on opened device Diederick Niehorster
2022-03-29  3:59 ` [FFmpeg-devel] [PATCH v4 00/22] avdevice (mostly dshow) enhancements Roger Pack

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20220325141041.1748-5-dcnieho@gmail.com \
    --to=dcnieho@gmail.com \
    --cc=ffmpeg-devel@ffmpeg.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://master.gitmailbox.com/ffmpegdev/0 ffmpegdev/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 ffmpegdev ffmpegdev/ https://master.gitmailbox.com/ffmpegdev \
		ffmpegdev@gitmailbox.com
	public-inbox-index ffmpegdev

Example config snippet for mirrors.


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git