Dave Reisner wrote:
> + else if (!quiet) {
> printf(_("%s is a mountpoint\n"), spec);
> + rc = EXIT_SUCCESS;
> + }
So mountpoint will never return SUCCESS if -q is used?
> - return rc ? EXIT_FAILURE : EXIT_SUCCESS;
> +finish:
> + return rc;
IMHO 'goto' is ugly and should only be used in more difficult
cases. In this case - if you really liked to avoid several
free(spec) statements - it could be easily done with a few
else statements.
Berny--
To unsubscribe from this list: send the line "unsubscribe util-linux" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
