After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 547518 - pulsemixer probe fails after device property is set
pulsemixer probe fails after device property is set
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-good
git master
Other All
: Normal normal
: 0.10.11
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2008-08-13 01:24 UTC by Laszlo Pandy
Modified: 2008-08-13 12:11 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Move the set_server to be called when the server property changes. (756 bytes, patch)
2008-08-13 01:25 UTC, Laszlo Pandy
committed Details | Review

Description Laszlo Pandy 2008-08-13 01:24:45 UTC
Please describe the problem:
On the pulsemixer element, the property probe on the "device" property functions correctly until the value of the "device" property is changed. Then it always returns an empty list when the values are probed.

You do not even need the test case so verify this. It is obvious from the code that the server property is being set when the device property is changed.

Steps to reproduce:
In a python shell:
>>> import gst
>>> pm = gst.element_factory_make("pulsemixer")
>>> pm.set_state(gst.STATE_READY)
>>> pm.probe_property_name("device")
>>> pm.probe_get_values_name("device")
['alsa_output.pci_10de_6a_sound_card_0_alsa_playback_0',
 'alsa_output.pci_10de_6a_sound_card_0_alsa_playback_0.monitor',
 'alsa_input.pci_10de_6a_sound_card_0_alsa_capture_0']
>>> pm.set_property("device", 'alsa_output.pci_10de_6a_sound_card_0_alsa_playback_0')
>>> pm.probe_property_name("device")
>>> pm.probe_get_values_name("device")
[]

Actual results:


Expected results:


Does this happen every time?


Other information:
Comment 1 Laszlo Pandy 2008-08-13 01:25:48 UTC
Created attachment 116468 [details] [review]
Move the set_server to be called when the server property changes.
Comment 2 Sebastian Dröge (slomo) 2008-08-13 12:11:21 UTC
2008-08-13  Sebastian Dröge  <sebastian.droege@collabora.co.uk>

        Patch by: Laszlo Pandy <laszlok2 at gmail dot com>

        * ext/pulse/pulsemixer.c: (gst_pulsemixer_set_property):
        Fix property probing after the device property is set by calling
        set_server when the server property changes. Fixes bug #547518.