[Spice-devel] [spice-gtk v1] gstaudio: Remove fixed TODOs about setting record volume

Victor Toso victortoso at redhat.com
Thu Mar 9 09:07:47 UTC 2017


From: Victor Toso <me at victortoso.com>

* directsoundsrc doesn't support IDirectSoundBuffer_SetVolume

This was fixed by 1b564bfb8b0db in gst-plugins-bad, see:
https://bugzilla.gnome.org/show_bug.cgi?id=744383

* pulsesrc doesn't support volume property, it's all coming!

This was fixed by 1f4bb687940f87a0 in gst-plugins-good, see:
https://bugzilla.gnome.org/show_bug.cgi?id=595055

Signed-off-by: Victor Toso <victortoso at redhat.com>
---
 src/spice-gstaudio.c | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/src/spice-gstaudio.c b/src/spice-gstaudio.c
index 1d96c77..65d66e3 100644
--- a/src/spice-gstaudio.c
+++ b/src/spice-gstaudio.c
@@ -430,9 +430,6 @@ static void record_volume_changed(GObject *object, GParamSpec *pspec, gpointer d
     vol = 1.0 * volume[0] / VOLUME_NORMAL;
     SPICE_DEBUG("record volume changed to %u (%0.2f)", volume[0], 100*vol);
 
-    /* TODO directsoundsrc doesn't support IDirectSoundBuffer_SetVolume */
-    /* TODO pulsesrc doesn't support volume property, it's all coming! */
-
     if (GST_IS_BIN(p->record.src))
         e = gst_bin_get_by_interface(GST_BIN(p->record.src), GST_TYPE_STREAM_VOLUME);
     else
-- 
2.9.3



More information about the Spice-devel mailing list