Fix various instance checks

The g_return*_if_fail() calls should use PREFIX_IS_TYPE_NAME [1], which
returns whether the given instance is of the right type, not
PREFIX_TYPE_NAME [2], which checks whether the given instance if of the
right type, warns if not, and returns the passed-in instance unchanged.

Using the wrong one means the returned value is the passed-in pointer,
and thus passes the g_return*_if_fail() whenever the pointer is
non-NULL, no matter whether it's of the right type or not.

A warning would still be emitted by g_type_check_instance_cast(), but
the execution wouldn't get short-circuited.

[1] calls g_type_check_instance_is_a()
[2] calls g_type_check_instance_cast()

1 files changed, 2 insertions, 2 deletions

diff --git a/mate-volume-control/gvc-stream-status-icon.c b/mate-volume-control/gvc-stream-status-icon.c
index b1371cf..dccd20d 100644
--- a/mate-volume-control/gvc-stream-status-icon.c
+++ b/mate-volume-control/gvc-stream-status-icon.c
@@ -513,7 +513,7 @@ void
 gvc_stream_status_icon_set_display_name (GvcStreamStatusIcon *icon,
                                          const gchar         *name)
 {
-        g_return_if_fail (GVC_STREAM_STATUS_ICON (icon));
+        g_return_if_fail (GVC_IS_STREAM_STATUS_ICON (icon));
 
         g_free (icon->priv->display_name);
 
@@ -527,7 +527,7 @@ void
 gvc_stream_status_icon_set_control (GvcStreamStatusIcon    *icon,
                                     MateMixerStreamControl *control)
 {
-        g_return_if_fail (GVC_STREAM_STATUS_ICON (icon));
+        g_return_if_fail (GVC_IS_STREAM_STATUS_ICON (icon));
 
         if (icon->priv->control == control)
                 return;