On 31.10.2017 14:29, Tanu Kaskinen wrote:
The filter sources should have the same max_rewind as the master source,
but these modules didn't update max_rewind when the master max_rewind
changed.
---
  src/modules/module-remap-source.c   | 12 ++++++++++++
  src/modules/module-virtual-source.c | 12 ++++++++++++
  2 files changed, 24 insertions(+)

diff --git a/src/modules/module-remap-source.c 
b/src/modules/module-remap-source.c
index 281d651f4..980f2554a 100644
--- a/src/modules/module-remap-source.c
+++ b/src/modules/module-remap-source.c
@@ -175,6 +175,17 @@ static void 
source_output_process_rewind_cb(pa_source_output *o, size_t nbytes)
          pa_source_process_rewind(u->source, nbytes);
  }
+/* Called from output thread context */
+static void source_output_update_max_rewind_cb(pa_source_output *o, size_t 
nbytes) {
+    struct userdata *u;
+
+    pa_source_output_assert_ref(o);
+    pa_source_output_assert_io_context(o);
+    pa_assert_se(u = o->userdata);
+
+    pa_source_set_max_rewind_within_thread(u->source, nbytes);
+}
+
  /* Called from output thread context */
  static void source_output_detach_cb(pa_source_output *o) {
      struct userdata *u;
@@ -387,6 +398,7 @@ int pa__init(pa_module*m) {
u->source_output->push = source_output_push_cb;
      u->source_output->process_rewind = source_output_process_rewind_cb;
+    u->source_output->update_max_rewind = source_output_update_max_rewind_cb;
      u->source_output->kill = source_output_kill_cb;
      u->source_output->attach = source_output_attach_cb;
      u->source_output->detach = source_output_detach_cb;
diff --git a/src/modules/module-virtual-source.c 
b/src/modules/module-virtual-source.c
index 94907dfa9..8ec4525a7 100644
--- a/src/modules/module-virtual-source.c
+++ b/src/modules/module-virtual-source.c
@@ -370,6 +370,17 @@ static void 
source_output_process_rewind_cb(pa_source_output *o, size_t nbytes)
  #endif
  }
+/* Called from output thread context */
+static void source_output_update_max_rewind_cb(pa_source_output *o, size_t 
nbytes) {
+    struct userdata *u;
+
+    pa_source_output_assert_ref(o);
+    pa_source_output_assert_io_context(o);
+    pa_assert_se(u = o->userdata);
+
+    pa_source_set_max_rewind_within_thread(u->source, nbytes);
+}
+
  /* Called from output thread context */
  static void source_output_attach_cb(pa_source_output *o) {
      struct userdata *u;
@@ -605,6 +616,7 @@ int pa__init(pa_module*m) {
u->source_output->push = source_output_push_cb;
      u->source_output->process_rewind = source_output_process_rewind_cb;
+    u->source_output->update_max_rewind = source_output_update_max_rewind_cb;
      u->source_output->kill = source_output_kill_cb;
      u->source_output->attach = source_output_attach_cb;
      u->source_output->detach = source_output_detach_cb;

LGTM

_______________________________________________
pulseaudio-discuss mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/pulseaudio-discuss

Reply via email to