[PATCH 5/8] sensord: Refactoring of parseChips()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Did some refactoring in parseChips(). This fixes some deep indentation
levels.
---

 args.c |   37 +++++++++++++++++++------------------
 1 file changed, 19 insertions(+), 18 deletions(-)

--- quilt-sensors.orig/prog/sensord/args.c	2009-03-24 22:01:55.000000000 +0100
+++ quilt-sensors/prog/sensord/args.c	2009-03-24 22:02:56.000000000 +0100
@@ -279,6 +279,8 @@
 
 int parseChips(int argc, char **argv)
 {
+	int i, n = argc - optind, err;
+
 	if (optind == argc) {
 		sensord_args.chipNames[0].prefix =
 			SENSORS_CHIP_NAME_PREFIX_ANY;
@@ -286,26 +288,25 @@
 		sensord_args.chipNames[0].bus.nr = SENSORS_BUS_NR_ANY;
 		sensord_args.chipNames[0].addr = SENSORS_CHIP_NAME_ADDR_ANY;
 		sensord_args.numChipNames = 1;
-	} else {
-		int i, n = argc - optind, err;
-		if (n > MAX_CHIP_NAMES) {
-			fprintf(stderr, "Too many chip names.\n");
-			return -1;
-		}
-		for (i = 0; i < n; ++ i) {
-			char *arg = argv[optind + i];
 
-			err = sensors_parse_chip_name(arg,
-						      sensord_args.chipNames +
-						      i);
-			if (err) {
-				fprintf(stderr,
-					"Invalid chip name `%s': %s\n", arg,
-					sensors_strerror(err));
-				return -1;
-			}
+		return 0;
+	}
+
+	if (n > MAX_CHIP_NAMES) {
+		fprintf(stderr, "Too many chip names.\n");
+		return -1;
+	}
+	for (i = 0; i < n; ++ i) {
+		char *arg = argv[optind + i];
+
+		err = sensors_parse_chip_name(arg, sensord_args.chipNames + i);
+		if (err) {
+			fprintf(stderr,	"Invalid chip name `%s': %s\n", arg,
+				sensors_strerror(err));
+			return -1;
 		}
-		sensord_args.numChipNames = n;
 	}
+	sensord_args.numChipNames = n;
+
 	return 0;
 }



[Index of Archives]     [Linux Kernel]     [Linux Hardware Monitoring]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux