summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMichal Ratajsky <[email protected]>2014-08-20 18:55:10 +0200
committerinfirit <[email protected]>2014-10-09 16:53:01 +0200
commit1f6448f52b7666a93aa28ec7dafd0d8944c45104 (patch)
tree9b435b659571d2edb479719ad4fc7eeff809cd03
parent18482b16107f4b5c610c83ddd366402efadd44d1 (diff)
downloadmate-media-1f6448f52b7666a93aa28ec7dafd0d8944c45104.tar.bz2
mate-media-1f6448f52b7666a93aa28ec7dafd0d8944c45104.tar.xz
Fix problems with setting current/default input/output stream
-rw-r--r--mate-volume-control/src/gvc-mixer-dialog.c69
1 files changed, 42 insertions, 27 deletions
diff --git a/mate-volume-control/src/gvc-mixer-dialog.c b/mate-volume-control/src/gvc-mixer-dialog.c
index 9132fa3..1393024 100644
--- a/mate-volume-control/src/gvc-mixer-dialog.c
+++ b/mate-volume-control/src/gvc-mixer-dialog.c
@@ -435,10 +435,25 @@ update_output_settings (GvcMixerDialog *dialog)
static void
set_output_stream (GvcMixerDialog *dialog, MateMixerStream *stream)
{
- GtkTreeModel *model;
- MateMixerSwitch *swtch;
+ GtkTreeModel *model;
+ MateMixerSwitch *swtch;
+ MateMixerStreamControl *control;
+
+ control = gvc_channel_bar_get_control (GVC_CHANNEL_BAR (dialog->priv->output_bar));
+ if (control != NULL) {
+ /* Disconnect port switch of the previous stream */
+ if (dialog->priv->output_port_combo != NULL) {
+ swtch = g_object_get_data (G_OBJECT (dialog->priv->output_port_combo),
+ "switch");
+ if (swtch != NULL)
+ g_signal_handlers_disconnect_by_data (G_OBJECT (swtch),
+ dialog);
+ }
+ }
+
+ bar_set_stream (dialog, dialog->priv->output_bar, stream);
- if (0 && stream != NULL) {
+ if (stream != NULL) {
const GList *controls;
controls = mate_mixer_context_list_stored_controls (dialog->priv->context);
@@ -451,7 +466,9 @@ set_output_stream (GvcMixerDialog *dialog, MateMixerStream *stream)
control = MATE_MIXER_STREAM_CONTROL (controls->data);
parent = mate_mixer_stream_control_get_stream (control);
- if (parent != stream) {
+ /* Prefer streamless controls to stay the way they are, forcing them to
+ * a particular owning stream would be wrong for eg. event controls */
+ if (parent != NULL && parent != stream) {
MateMixerDirection direction =
mate_mixer_stream_get_direction (parent);
@@ -462,15 +479,6 @@ set_output_stream (GvcMixerDialog *dialog, MateMixerStream *stream)
}
}
- /* Changing the default input stream, disconnect the port switch */
- if (dialog->priv->output_port_combo != NULL) {
- swtch = g_object_get_data (G_OBJECT (dialog->priv->output_port_combo), "switch");
- if (swtch != NULL)
- g_signal_handlers_disconnect_by_data (G_OBJECT (swtch), dialog);
- }
-
- bar_set_stream (dialog, dialog->priv->output_bar, stream);
-
model = gtk_tree_view_get_model (GTK_TREE_VIEW (dialog->priv->output_treeview));
update_default_tree_item (dialog, model, stream);
@@ -597,7 +605,16 @@ set_input_stream (GvcMixerDialog *dialog, MateMixerStream *stream)
control = gvc_channel_bar_get_control (GVC_CHANNEL_BAR (dialog->priv->input_bar));
if (control != NULL) {
- /* Make sure to disable monitoring of the removed stream */
+ /* Disconnect port switch of the previous stream */
+ if (dialog->priv->input_port_combo != NULL) {
+ swtch = g_object_get_data (G_OBJECT (dialog->priv->input_port_combo),
+ "switch");
+ if (swtch != NULL)
+ g_signal_handlers_disconnect_by_data (G_OBJECT (swtch),
+ dialog);
+ }
+
+ /* Disable monitoring of the previous control */
g_signal_handlers_disconnect_by_func (G_OBJECT (control),
G_CALLBACK (on_stream_control_monitor_value),
dialog);
@@ -605,6 +622,8 @@ set_input_stream (GvcMixerDialog *dialog, MateMixerStream *stream)
mate_mixer_stream_control_set_monitor_enabled (control, FALSE);
}
+ bar_set_stream (dialog, dialog->priv->input_bar, stream);
+
if (stream != NULL) {
const GList *controls;
guint page = gtk_notebook_get_current_page (GTK_NOTEBOOK (dialog->priv->notebook));
@@ -618,7 +637,9 @@ set_input_stream (GvcMixerDialog *dialog, MateMixerStream *stream)
control = MATE_MIXER_STREAM_CONTROL (controls->data);
parent = mate_mixer_stream_control_get_stream (control);
- if (parent != stream) {
+ /* Prefer streamless controls to stay the way they are, forcing them to
+ * a particular owning stream would be wrong for eg. event controls */
+ if (parent != NULL && parent != stream) {
MateMixerDirection direction =
mate_mixer_stream_get_direction (parent);
@@ -628,20 +649,14 @@ set_input_stream (GvcMixerDialog *dialog, MateMixerStream *stream)
controls = controls->next;
}
- if (page == PAGE_INPUT)
- mate_mixer_stream_control_set_monitor_enabled (control, TRUE);
- }
-
- /* Changing the default input stream, disconnect the port switch */
- if (dialog->priv->input_port_combo != NULL) {
- swtch = g_object_get_data (G_OBJECT (dialog->priv->input_port_combo), "switch");
- if (swtch != NULL)
- g_signal_handlers_disconnect_by_data (G_OBJECT (swtch), dialog);
- }
+ if (page == PAGE_INPUT) {
+ MateMixerStreamControl *control =
+ gvc_channel_bar_get_control (GVC_CHANNEL_BAR (dialog->priv->input_bar));
- bar_set_stream (dialog, dialog->priv->input_bar, stream);
+ if (G_LIKELY (control != NULL))
+ mate_mixer_stream_control_set_monitor_enabled (control, TRUE);
+ }
- if (stream != NULL) {
/* Enable/disable the peak level monitor according to mute state */
g_signal_connect (G_OBJECT (stream),
"notify::mute",