timer: instance-params: add checks for method arguments

In Rules for use of GError, it's just used for recoverable runtime
error, not for programming error. The invalid arguments are a kind of
programming error.

This commit adds the check for method arguments.

Signed-off-by: Takashi Sakamoto <o-takashi@sakamocchi.jp>

diff --git a/src/timer/instance-params.c b/src/timer/instance-params.c
index 37fa373424ee941e38ef5a32fcf920ca8804fa7a..b19382ccf074230b4437769cd9f9927c5ee8b6af 100644 (file)
--- a/src/timer/instance-params.c
+++ b/src/timer/instance-params.c
@@ -165,8 +165,7 @@ void alsatimer_instance_params_set_event_filter(ALSATimerInstanceParams *self,
             (val > SNDRV_TIMER_EVENT_RESUME &&
              val < SNDRV_TIMER_EVENT_MSTART) ||
             val > SNDRV_TIMER_EVENT_MRESUME) {
-            generate_error(error, EINVAL);
-            return;
+            g_return_if_reached();
         }
         filter |= (1u << val);
     }
@@ -225,9 +224,8 @@ void alsatimer_instance_params_get_event_filter(ALSATimerInstanceParams *self,
     }
 
     if (index != count) {
-        generate_error(error, ENXIO);
         g_free(list);
-        return;
+        g_return_if_reached();
     }
 
     *entries = list;