From: Felix Fietkau <nbd@openwrt.org>
Date: Sat, 2 Feb 2008 22:05:16 +0000 (+0100)
Subject: optimization
X-Git-Url: http://git.archive.openwrt.org/?p=project%2Fuci.git;a=commitdiff_plain;h=05cf670dde3610279a07b76277cc29ceb4d7ad00;ds=sidebyside

optimization
---

diff --git a/cli.c b/cli.c
index 0d663e1..87de48c 100644
--- a/cli.c
+++ b/cli.c
@@ -137,13 +137,14 @@ static int uci_do_package_cmd(int cmd, int argc, char **argv)
 
 static int uci_do_section_cmd(int cmd, int argc, char **argv)
 {
+	struct uci_package *p = NULL;
+	struct uci_element *e = NULL;
 	char *package = NULL;
 	char *section = NULL;
 	char *option = NULL;
 	char *value = NULL;
 	char **ptr = NULL;
-	struct uci_package *p = NULL;
-	struct uci_element *e = NULL;
+	int ret = UCI_OK;
 
 	if (argc != 2)
 		return 255;
@@ -184,22 +185,13 @@ static int uci_do_section_cmd(int cmd, int argc, char **argv)
 		printf("%s\n", value);
 		break;
 	case CMD_RENAME:
-		if (uci_rename(ctx, p, section, option, value) != UCI_OK) {
-			uci_perror(ctx, appname);
-			return 1;
-		}
+		ret = uci_rename(ctx, p, section, option, value);
 		break;
 	case CMD_SET:
-		if (uci_set(ctx, p, section, option, value) != UCI_OK) {
-			uci_perror(ctx, appname);
-			return 1;
-		}
+		ret = uci_set(ctx, p, section, option, value);
 		break;
 	case CMD_DEL:
-		if (uci_delete(ctx, p, section, option) != UCI_OK) {
-			uci_perror(ctx, appname);
-			return 1;
-		}
+		ret = uci_delete(ctx, p, section, option);
 		break;
 	}
 
@@ -208,7 +200,10 @@ static int uci_do_section_cmd(int cmd, int argc, char **argv)
 		return 0;
 
 	/* save changes, but don't commit them yet */
-	if (uci_save(ctx, p) != UCI_OK) {
+	if (ret == UCI_OK)
+		ret = uci_save(ctx, p);
+
+	if (ret != UCI_OK) {
 		uci_perror(ctx, appname);
 		return 1;
 	}