diff options
| author | Michal Ratajsky <[email protected]> | 2014-10-31 14:50:29 +0100 |
|---|---|---|
| committer | Michal Ratajsky <[email protected]> | 2014-10-31 14:50:29 +0100 |
| commit | 20afadff90fb6a866d3e361b2b294a67775da95e (patch) | |
| tree | f5707583cb7fbe267bdba4c04a639e8e9b8dfb5c /libmatemixer/matemixer-switch.c | |
| parent | 1897445a80677ec89e7eef9f2a76e630fd088588 (diff) | |
| download | libmatemixer-20afadff90fb6a866d3e361b2b294a67775da95e.tar.bz2 libmatemixer-20afadff90fb6a866d3e361b2b294a67775da95e.tar.xz | |
Remove ability to have no active switch option and fix OSS to correctly handle such case
Diffstat (limited to 'libmatemixer/matemixer-switch.c')
| -rw-r--r-- | libmatemixer/matemixer-switch.c | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/libmatemixer/matemixer-switch.c b/libmatemixer/matemixer-switch.c index 87580dd..6466838 100644 --- a/libmatemixer/matemixer-switch.c +++ b/libmatemixer/matemixer-switch.c @@ -359,6 +359,7 @@ mate_mixer_switch_set_active_option (MateMixerSwitch *swtch, MateMixerSwitchClass *klass; g_return_val_if_fail (MATE_MIXER_IS_SWITCH (swtch), FALSE); + g_return_val_if_fail (MATE_MIXER_IS_SWITCH_OPTION (option), FALSE); klass = MATE_MIXER_SWITCH_GET_CLASS (swtch); if (klass->set_active_option == NULL) @@ -398,7 +399,7 @@ _mate_mixer_switch_set_active_option (MateMixerSwitch *swtch, MateMixerSwitchOption *option) { g_return_if_fail (MATE_MIXER_IS_SWITCH (swtch)); - g_return_if_fail (option == NULL || MATE_MIXER_IS_SWITCH_OPTION (option)); + g_return_if_fail (MATE_MIXER_IS_SWITCH_OPTION (option)); if (swtch->priv->active == option) return; @@ -406,10 +407,7 @@ _mate_mixer_switch_set_active_option (MateMixerSwitch *swtch, if (swtch->priv->active != NULL) g_object_unref (swtch->priv->active); - if (option != NULL) - swtch->priv->active = g_object_ref (option); - else - swtch->priv->active = NULL; + swtch->priv->active = g_object_ref (option); g_object_notify_by_pspec (G_OBJECT (swtch), properties[PROP_ACTIVE_OPTION]); } |