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; }