kerneldevice: remove all unneeded g_return_if_fail() checks

These kind of checks are only useful on public APIs really, there
should be no need to have them in internal code.

1 files changed, 0 insertions, 57 deletions

diff --git a/src/kerneldevice/mm-kernel-device.c b/src/kerneldevice/mm-kernel-device.c
index cf45447f..71e13077 100644
--- a/src/kerneldevice/mm-kernel-device.c
+++ b/src/kerneldevice/mm-kernel-device.c
@@ -29,8 +29,6 @@ G_DEFINE_ABSTRACT_TYPE_WITH_CODE (MMKernelDevice, mm_kernel_device, G_TYPE_OBJEC
 const gchar *
 mm_kernel_device_get_subsystem (MMKernelDevice *self)
 {
-    g_return_val_if_fail (MM_IS_KERNEL_DEVICE (self), NULL);
-
     return (MM_KERNEL_DEVICE_GET_CLASS (self)->get_subsystem ?
             MM_KERNEL_DEVICE_GET_CLASS (self)->get_subsystem (self) :
             NULL);
@@ -39,8 +37,6 @@ mm_kernel_device_get_subsystem (MMKernelDevice *self)
 const gchar *
 mm_kernel_device_get_name (MMKernelDevice *self)
 {
-    g_return_val_if_fail (MM_IS_KERNEL_DEVICE (self), NULL);
-
     return (MM_KERNEL_DEVICE_GET_CLASS (self)->get_name ?
             MM_KERNEL_DEVICE_GET_CLASS (self)->get_name (self) :
             NULL);
@@ -49,8 +45,6 @@ mm_kernel_device_get_name (MMKernelDevice *self)
 const gchar *
 mm_kernel_device_get_driver (MMKernelDevice *self)
 {
-    g_return_val_if_fail (MM_IS_KERNEL_DEVICE (self), NULL);
-
     return (MM_KERNEL_DEVICE_GET_CLASS (self)->get_driver ?
             MM_KERNEL_DEVICE_GET_CLASS (self)->get_driver (self) :
             NULL);
@@ -59,8 +53,6 @@ mm_kernel_device_get_driver (MMKernelDevice *self)
 const gchar *
 mm_kernel_device_get_sysfs_path (MMKernelDevice *self)
 {
-    g_return_val_if_fail (MM_IS_KERNEL_DEVICE (self), NULL);
-
     return (MM_KERNEL_DEVICE_GET_CLASS (self)->get_sysfs_path ?
             MM_KERNEL_DEVICE_GET_CLASS (self)->get_sysfs_path (self) :
             NULL);
@@ -69,8 +61,6 @@ mm_kernel_device_get_sysfs_path (MMKernelDevice *self)
 const gchar *
 mm_kernel_device_get_physdev_uid (MMKernelDevice *self)
 {
-    g_return_val_if_fail (MM_IS_KERNEL_DEVICE (self), NULL);
-
     return (MM_KERNEL_DEVICE_GET_CLASS (self)->get_physdev_uid ?
             MM_KERNEL_DEVICE_GET_CLASS (self)->get_physdev_uid (self) :
             NULL);
@@ -79,8 +69,6 @@ mm_kernel_device_get_physdev_uid (MMKernelDevice *self)
 guint16
 mm_kernel_device_get_physdev_vid (MMKernelDevice *self)
 {
-    g_return_val_if_fail (MM_IS_KERNEL_DEVICE (self), 0);
-
     return (MM_KERNEL_DEVICE_GET_CLASS (self)->get_physdev_vid ?
             MM_KERNEL_DEVICE_GET_CLASS (self)->get_physdev_vid (self) :
             0);
@@ -89,8 +77,6 @@ mm_kernel_device_get_physdev_vid (MMKernelDevice *self)
 guint16
 mm_kernel_device_get_physdev_pid (MMKernelDevice *self)
 {
-    g_return_val_if_fail (MM_IS_KERNEL_DEVICE (self), 0);
-
     return (MM_KERNEL_DEVICE_GET_CLASS (self)->get_physdev_pid ?
             MM_KERNEL_DEVICE_GET_CLASS (self)->get_physdev_pid (self) :
             0);
@@ -99,8 +85,6 @@ mm_kernel_device_get_physdev_pid (MMKernelDevice *self)
 guint16
 mm_kernel_device_get_physdev_revision (MMKernelDevice *self)
 {
-    g_return_val_if_fail (MM_IS_KERNEL_DEVICE (self), 0);
-
     return (MM_KERNEL_DEVICE_GET_CLASS (self)->get_physdev_revision ?
             MM_KERNEL_DEVICE_GET_CLASS (self)->get_physdev_revision (self) :
             0);
@@ -109,8 +93,6 @@ mm_kernel_device_get_physdev_revision (MMKernelDevice *self)
 const gchar *
 mm_kernel_device_get_physdev_subsystem (MMKernelDevice *self)
 {
-    g_return_val_if_fail (MM_IS_KERNEL_DEVICE (self), NULL);
-
     return (MM_KERNEL_DEVICE_GET_CLASS (self)->get_physdev_subsystem ?
             MM_KERNEL_DEVICE_GET_CLASS (self)->get_physdev_subsystem (self) :
             NULL);
@@ -119,8 +101,6 @@ mm_kernel_device_get_physdev_subsystem (MMKernelDevice *self)
 const gchar *
 mm_kernel_device_get_physdev_sysfs_path (MMKernelDevice *self)
 {
-    g_return_val_if_fail (MM_IS_KERNEL_DEVICE (self), NULL);
-
     return (MM_KERNEL_DEVICE_GET_CLASS (self)->get_physdev_sysfs_path ?
             MM_KERNEL_DEVICE_GET_CLASS (self)->get_physdev_sysfs_path (self) :
             NULL);
@@ -129,8 +109,6 @@ mm_kernel_device_get_physdev_sysfs_path (MMKernelDevice *self)
 const gchar *
 mm_kernel_device_get_physdev_manufacturer (MMKernelDevice *self)
 {
-    g_return_val_if_fail (MM_IS_KERNEL_DEVICE (self), NULL);
-
     return (MM_KERNEL_DEVICE_GET_CLASS (self)->get_physdev_manufacturer ?
             MM_KERNEL_DEVICE_GET_CLASS (self)->get_physdev_manufacturer (self) :
             NULL);
@@ -139,8 +117,6 @@ mm_kernel_device_get_physdev_manufacturer (MMKernelDevice *self)
 const gchar *
 mm_kernel_device_get_physdev_product (MMKernelDevice *self)
 {
-    g_return_val_if_fail (MM_IS_KERNEL_DEVICE (self), NULL);
-
     return (MM_KERNEL_DEVICE_GET_CLASS (self)->get_physdev_product ?
             MM_KERNEL_DEVICE_GET_CLASS (self)->get_physdev_product (self) :
             NULL);
@@ -149,8 +125,6 @@ mm_kernel_device_get_physdev_product (MMKernelDevice *self)
 gint
 mm_kernel_device_get_interface_class (MMKernelDevice *self)
 {
-    g_return_val_if_fail (MM_IS_KERNEL_DEVICE (self), -1);
-
     return (MM_KERNEL_DEVICE_GET_CLASS (self)->get_interface_class ?
             MM_KERNEL_DEVICE_GET_CLASS (self)->get_interface_class (self) :
             -1);
@@ -159,8 +133,6 @@ mm_kernel_device_get_interface_class (MMKernelDevice *self)
 gint
 mm_kernel_device_get_interface_subclass (MMKernelDevice *self)
 {
-    g_return_val_if_fail (MM_IS_KERNEL_DEVICE (self), -1);
-
     return (MM_KERNEL_DEVICE_GET_CLASS (self)->get_interface_subclass ?
             MM_KERNEL_DEVICE_GET_CLASS (self)->get_interface_subclass (self) :
             -1);
@@ -169,8 +141,6 @@ mm_kernel_device_get_interface_subclass (MMKernelDevice *self)
 gint
 mm_kernel_device_get_interface_protocol (MMKernelDevice *self)
 {
-    g_return_val_if_fail (MM_IS_KERNEL_DEVICE (self), -1);
-
     return (MM_KERNEL_DEVICE_GET_CLASS (self)->get_interface_protocol ?
             MM_KERNEL_DEVICE_GET_CLASS (self)->get_interface_protocol (self) :
             -1);
@@ -179,8 +149,6 @@ mm_kernel_device_get_interface_protocol (MMKernelDevice *self)
 const gchar *
 mm_kernel_device_get_interface_sysfs_path (MMKernelDevice *self)
 {
-    g_return_val_if_fail (MM_IS_KERNEL_DEVICE (self), NULL);
-
     return (MM_KERNEL_DEVICE_GET_CLASS (self)->get_interface_sysfs_path ?
             MM_KERNEL_DEVICE_GET_CLASS (self)->get_interface_sysfs_path (self) :
             NULL);
@@ -189,8 +157,6 @@ mm_kernel_device_get_interface_sysfs_path (MMKernelDevice *self)
 const gchar *
 mm_kernel_device_get_interface_description (MMKernelDevice *self)
 {
-    g_return_val_if_fail (MM_IS_KERNEL_DEVICE (self), NULL);
-
     return (MM_KERNEL_DEVICE_GET_CLASS (self)->get_interface_description ?
             MM_KERNEL_DEVICE_GET_CLASS (self)->get_interface_description (self) :
             NULL);
@@ -200,9 +166,6 @@ gboolean
 mm_kernel_device_cmp (MMKernelDevice *a,
                       MMKernelDevice *b)
 {
-    g_return_val_if_fail (MM_IS_KERNEL_DEVICE (a), FALSE);
-    g_return_val_if_fail (MM_IS_KERNEL_DEVICE (b), FALSE);
-
     if (G_OBJECT_TYPE (a) != G_OBJECT_TYPE (b))
         return FALSE;
 
@@ -215,8 +178,6 @@ gboolean
 mm_kernel_device_has_property (MMKernelDevice *self,
                                const gchar    *property)
 {
-    g_return_val_if_fail (MM_IS_KERNEL_DEVICE (self), FALSE);
-
     return (MM_KERNEL_DEVICE_GET_CLASS (self)->has_property ?
             MM_KERNEL_DEVICE_GET_CLASS (self)->has_property (self, property) :
             FALSE);
@@ -226,8 +187,6 @@ const gchar *
 mm_kernel_device_get_property (MMKernelDevice *self,
                                const gchar    *property)
 {
-    g_return_val_if_fail (MM_IS_KERNEL_DEVICE (self), NULL);
-
     return (MM_KERNEL_DEVICE_GET_CLASS (self)->get_property ?
             MM_KERNEL_DEVICE_GET_CLASS (self)->get_property (self, property) :
             NULL);
@@ -237,8 +196,6 @@ gboolean
 mm_kernel_device_get_property_as_boolean (MMKernelDevice *self,
                                           const gchar    *property)
 {
-    g_return_val_if_fail (MM_IS_KERNEL_DEVICE (self), FALSE);
-
     return (MM_KERNEL_DEVICE_GET_CLASS (self)->get_property_as_boolean ?
             MM_KERNEL_DEVICE_GET_CLASS (self)->get_property_as_boolean (self, property) :
             FALSE);
@@ -248,8 +205,6 @@ gint
 mm_kernel_device_get_property_as_int (MMKernelDevice *self,
                                       const gchar    *property)
 {
-    g_return_val_if_fail (MM_IS_KERNEL_DEVICE (self), -1);
-
     return (MM_KERNEL_DEVICE_GET_CLASS (self)->get_property_as_int ?
             MM_KERNEL_DEVICE_GET_CLASS (self)->get_property_as_int (self, property) :
             -1);
@@ -259,8 +214,6 @@ guint
 mm_kernel_device_get_property_as_int_hex (MMKernelDevice *self,
                                           const gchar    *property)
 {
-    g_return_val_if_fail (MM_IS_KERNEL_DEVICE (self), 0);
-
     return (MM_KERNEL_DEVICE_GET_CLASS (self)->get_property_as_int_hex ?
             MM_KERNEL_DEVICE_GET_CLASS (self)->get_property_as_int_hex (self, property) :
             0);
@@ -270,8 +223,6 @@ gboolean
 mm_kernel_device_has_global_property (MMKernelDevice *self,
                                       const gchar    *property)
 {
-    g_return_val_if_fail (MM_IS_KERNEL_DEVICE (self), FALSE);
-
     return (MM_KERNEL_DEVICE_GET_CLASS (self)->has_global_property ?
             MM_KERNEL_DEVICE_GET_CLASS (self)->has_global_property (self, property) :
             FALSE);
@@ -281,8 +232,6 @@ const gchar *
 mm_kernel_device_get_global_property (MMKernelDevice *self,
                                       const gchar    *property)
 {
-    g_return_val_if_fail (MM_IS_KERNEL_DEVICE (self), NULL);
-
     return (MM_KERNEL_DEVICE_GET_CLASS (self)->get_global_property ?
             MM_KERNEL_DEVICE_GET_CLASS (self)->get_global_property (self, property) :
             NULL);
@@ -292,8 +241,6 @@ gboolean
 mm_kernel_device_get_global_property_as_boolean (MMKernelDevice *self,
                                                  const gchar    *property)
 {
-    g_return_val_if_fail (MM_IS_KERNEL_DEVICE (self), FALSE);
-
     return (MM_KERNEL_DEVICE_GET_CLASS (self)->get_global_property_as_boolean ?
             MM_KERNEL_DEVICE_GET_CLASS (self)->get_global_property_as_boolean (self, property) :
             FALSE);
@@ -303,8 +250,6 @@ gint
 mm_kernel_device_get_global_property_as_int (MMKernelDevice *self,
                                              const gchar    *property)
 {
-    g_return_val_if_fail (MM_IS_KERNEL_DEVICE (self), -1);
-
     return (MM_KERNEL_DEVICE_GET_CLASS (self)->get_global_property_as_int ?
             MM_KERNEL_DEVICE_GET_CLASS (self)->get_global_property_as_int (self, property) :
             -1);
@@ -314,8 +259,6 @@ guint
 mm_kernel_device_get_global_property_as_int_hex (MMKernelDevice *self,
                                                  const gchar    *property)
 {
-    g_return_val_if_fail (MM_IS_KERNEL_DEVICE (self), 0);
-
     return (MM_KERNEL_DEVICE_GET_CLASS (self)->get_global_property_as_int_hex ?
             MM_KERNEL_DEVICE_GET_CLASS (self)->get_global_property_as_int_hex (self, property) :
             0);