From: Krzysztof Opasiak <k.opasiak@samsung.com>
Date: Mon, 22 Sep 2014 10:30:03 +0000 (+0200)
Subject: libusbgx: Return suitable error codes instead of -1
X-Git-Tag: libusbgx-v0.1.0~155
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=22ad8219e6ec22ebff8546265e0d1bcd280c7dfc;p=platform%2Fupstream%2Flibusbg.git

libusbgx: Return suitable error codes instead of -1

Signed-off-by: Krzysztof Opasiak <k.opasiak@samsung.com>
---

diff --git a/src/usbg.c b/src/usbg.c
index cd2179e..4d74a0d 100644
--- a/src/usbg.c
+++ b/src/usbg.c
@@ -316,18 +316,15 @@ static int usbg_lookup_function_type(const char *name)
 	int max = sizeof(function_names)/sizeof(char *);
 
 	if (!name)
-		return -1;
+		return USBG_ERROR_INVALID_PARAM;
 
 	do {
 		if (!strcmp(name, function_names[i]))
-			break;
+			return i;
 		i++;
 	} while (i != max);
 
-	if (i == max)
-		i = -1;
-
-	return i;
+	return USBG_ERROR_NOT_FOUND;
 }
 
 const const char *usbg_get_function_type_str(usbg_function_type type)