commit 98c1f16cbd8fd2e94a6be0db492b378d34a29310
author Simon Wilson <simonwilson@google.com> Tue Jun 07 16:12:32 2011 -0700
committer Simon Wilson <simonwilson@google.com> Tue Jun 07 16:14:21 2011 -0700
tree 05b0eb2a0c1dfabe875aacca853d28a4c835d024
parent b9d4f6bfee008efe1c38eea80fdabf1886c26cf0

Fix error cases and some spurious whitespace

mixer.c

diff --git a/mixer.c b/mixer.c
index 2bb37a1..0c2e233 100644
--- a/mixer.c
+++ b/mixer.c
@@ -169,7 +169,7 @@
 {
     if (!mixer) {
         errno = EINVAL;
-        return -1;
+        return 0;
     }
 
     return mixer->count;
@@ -177,14 +177,10 @@
 
 struct mixer_ctl *mixer_get_ctl(struct mixer *mixer, unsigned int id)
 {
-    if (!mixer) {
-        errno = EINVAL;
-        return NULL;
-    }
-
-    if (id < mixer->count)
+    if (mixer && (id < mixer->count))
         return mixer->ctl + id;
 
+    errno = EINVAL;
     return NULL;
 }
 
@@ -192,15 +188,15 @@
 {
     unsigned int n;
 
-    if (!mixer) {
-        errno = EINVAL;
-        return NULL;
-    }
+    if (!mixer)
+        goto error;
 
     for (n = 0; n < mixer->count; n++)
         if (!strcmp(name, (char*) mixer->info[n].id.name))
             return mixer->ctl + n;
 
+error:
+    errno = EINVAL;
     return NULL;
 }
 
@@ -264,24 +260,24 @@
 static int percent_to_int(struct snd_ctl_elem_info *ei, int percent)
 {
     int range;
-    
+
     if (percent > 100)
         percent = 100;
     else if (percent < 0)
         percent = 0;
-    
+
     range = (ei->value.integer.max - ei->value.integer.min);
-    
+
     return ei->value.integer.min + (range * percent) / 100;
 }
 
 static int int_to_percent(struct snd_ctl_elem_info *ei, int value)
 {
     int range = (ei->value.integer.max - ei->value.integer.min);
-    
+
     if (range == 0)
         return 0;
-    
+
     return ((value - ei->value.integer.min) / range) * 100;
 }