From 13cb8427736298c80055ecd6a9abbef45849b527 Mon Sep 17 00:00:00 2001 From: Matt Kraai Date: Mon, 30 Jul 2001 14:43:20 +0000 Subject: [PATCH] Rework option handling to save space. --- rdate.c | 30 +++++++++++------------------- util-linux/rdate.c | 30 +++++++++++------------------- 2 files changed, 22 insertions(+), 38 deletions(-) diff --git a/rdate.c b/rdate.c index 8deb35d14..50be4de8c 100644 --- a/rdate.c +++ b/rdate.c @@ -48,10 +48,10 @@ static time_t askremotedate(const char *host) h = xgethostbyname(host); /* get the IP addr */ if ((tserv = getservbyname("time", "tcp")) == NULL) /* find port # */ - perror_msg_and_die("%s", "time"); + perror_msg_and_die("time"); if ((fd = socket(AF_INET, SOCK_STREAM, 0)) < 0) /* get net connection */ - perror_msg_and_die("%s", "socket"); + perror_msg_and_die("socket"); memcpy(&s_in.sin_addr, h->h_addr, sizeof(s_in.sin_addr)); s_in.sin_port= tserv->s_port; @@ -80,33 +80,25 @@ int rdate_main(int argc, char **argv) { time_t remote_time; int opt; - int setdate = 0; - int printdate= 0; + int setdate = 1; + int printdate = 1; /* Interpret command line args */ - /* do special-case option parsing */ - if (argv[1] && (strcmp(argv[1], "--help") == 0)) - show_usage(); - - /* do normal option parsing */ - while ((opt = getopt(argc, argv, "Hsp")) > 0) { + while ((opt = getopt(argc, argv, "sp")) > 0) { switch (opt) { - default: - case 'H': - show_usage(); - break; case 's': - setdate++; + printdate = 0; + setdate = 1; break; case 'p': - printdate++; + printdate = 1; + setdate = 0; break; + default: + show_usage(); } } - /* the default action is to set the date */ - if (printdate==0 && setdate==0) setdate++; - if (optind == argc) show_usage(); diff --git a/util-linux/rdate.c b/util-linux/rdate.c index 8deb35d14..50be4de8c 100644 --- a/util-linux/rdate.c +++ b/util-linux/rdate.c @@ -48,10 +48,10 @@ static time_t askremotedate(const char *host) h = xgethostbyname(host); /* get the IP addr */ if ((tserv = getservbyname("time", "tcp")) == NULL) /* find port # */ - perror_msg_and_die("%s", "time"); + perror_msg_and_die("time"); if ((fd = socket(AF_INET, SOCK_STREAM, 0)) < 0) /* get net connection */ - perror_msg_and_die("%s", "socket"); + perror_msg_and_die("socket"); memcpy(&s_in.sin_addr, h->h_addr, sizeof(s_in.sin_addr)); s_in.sin_port= tserv->s_port; @@ -80,33 +80,25 @@ int rdate_main(int argc, char **argv) { time_t remote_time; int opt; - int setdate = 0; - int printdate= 0; + int setdate = 1; + int printdate = 1; /* Interpret command line args */ - /* do special-case option parsing */ - if (argv[1] && (strcmp(argv[1], "--help") == 0)) - show_usage(); - - /* do normal option parsing */ - while ((opt = getopt(argc, argv, "Hsp")) > 0) { + while ((opt = getopt(argc, argv, "sp")) > 0) { switch (opt) { - default: - case 'H': - show_usage(); - break; case 's': - setdate++; + printdate = 0; + setdate = 1; break; case 'p': - printdate++; + printdate = 1; + setdate = 0; break; + default: + show_usage(); } } - /* the default action is to set the date */ - if (printdate==0 && setdate==0) setdate++; - if (optind == argc) show_usage(); -- 2.25.1