Tito pointed out that if readlink doesn't have -f, there's no need to call

the option parsing code at all.
This commit is contained in:
Rob Landley 2005-09-17 11:13:25 +00:00
parent a882126fee
commit 07e42dc056

View File

@ -18,16 +18,16 @@
int readlink_main(int argc, char **argv) int readlink_main(int argc, char **argv)
{ {
char *buf; char *buf;
unsigned long opt = bb_getopt_ulflags(argc, argv, unsigned long opt = ENABLE_FEATURE_READLINK_FOLLOW ?
ENABLE_FEATURE_READLINK_FOLLOW ? "f" : ""); bb_getopt_ulflags(argc, argv, "f") : 0;
if (optind + 1 != argc) if (argc != (ENABLE_FEATURE_READLINK_FOLLOW ? optind + 1 : 2))
bb_show_usage(); bb_show_usage();
if (ENABLE_FEATURE_READLINK_FOLLOW && (opt & READLINK_FLAG_f)) if (opt & READLINK_FLAG_f)
buf = realpath(argv[optind], NULL); buf = realpath(argv[optind], NULL);
else else
buf = xreadlink(argv[optind]); buf = xreadlink(argv[ENABLE_FEATURE_READLINK_FOLLOW ? optind : 1]);
if (!buf) if (!buf)
return EXIT_FAILURE; return EXIT_FAILURE;