diff options
author | jkh <jkh@FreeBSD.org> | 1999-07-02 22:36:14 +0000 |
---|---|---|
committer | jkh <jkh@FreeBSD.org> | 1999-07-02 22:36:14 +0000 |
commit | 09d2fcf7ce9c557990687ca35240e537d4bd3ce3 (patch) | |
tree | bd8d251c407636bd37bfe867581ad952c8609cf5 /usr.sbin | |
parent | f83d7679292bf0c56becaeb56d8fff81d2cec1c5 (diff) | |
download | FreeBSD-src-09d2fcf7ce9c557990687ca35240e537d4bd3ce3.zip FreeBSD-src-09d2fcf7ce9c557990687ca35240e537d4bd3ce3.tar.gz |
Eliminate some varargs abuse.
Diffstat (limited to 'usr.sbin')
-rw-r--r-- | usr.sbin/sade/msg.c | 16 | ||||
-rw-r--r-- | usr.sbin/sade/sade.h | 3 | ||||
-rw-r--r-- | usr.sbin/sysinstall/index.c | 18 | ||||
-rw-r--r-- | usr.sbin/sysinstall/msg.c | 16 | ||||
-rw-r--r-- | usr.sbin/sysinstall/package.c | 18 | ||||
-rw-r--r-- | usr.sbin/sysinstall/sysinstall.h | 3 |
6 files changed, 30 insertions, 44 deletions
diff --git a/usr.sbin/sade/msg.c b/usr.sbin/sade/msg.c index a7116e9..7ee6d82 100644 --- a/usr.sbin/sade/msg.c +++ b/usr.sbin/sade/msg.c @@ -4,7 +4,7 @@ * This is probably the last program in the `sysinstall' line - the next * generation being essentially a complete rewrite. * - * $Id: msg.c,v 1.47 1997/09/20 02:48:48 jkh Exp $ + * $Id: msg.c,v 1.48 1999/05/27 10:32:48 jkh Exp $ * * Copyright (c) 1995 * Jordan Hubbard. All rights reserved. @@ -217,20 +217,6 @@ msgNotify(char *fmt, ...) dialog_msgbox(NULL, errstr, -1, -1, 0); } -/* Put up a message in a popup or notifier box, depending on NO_CONFIRM */ -void -msgCNotify(char *fmt, ...) -{ - va_list args; - - va_start(args, fmt); - if (variable_get(VAR_NO_CONFIRM)) - msgNotify(fmt, args); - else - msgConfirm(fmt, args); - va_end(args); -} - /* Put up a message in a popup yes/no box and return 1 for YES, 0 for NO */ int msgYesNo(char *fmt, ...) diff --git a/usr.sbin/sade/sade.h b/usr.sbin/sade/sade.h index c2ad3518..4816712 100644 --- a/usr.sbin/sade/sade.h +++ b/usr.sbin/sade/sade.h @@ -4,7 +4,7 @@ * This is probably the last attempt in the `sysinstall' line, the next * generation being slated to essentially a complete rewrite. * - * $Id: sysinstall.h,v 1.165 1999/05/15 14:34:22 jkh Exp $ + * $Id: sysinstall.h,v 1.166 1999/05/27 10:32:50 jkh Exp $ * * Copyright (c) 1995 * Jordan Hubbard. All rights reserved. @@ -645,7 +645,6 @@ extern void msgError(char *fmt, ...); extern void msgFatal(char *fmt, ...); extern void msgConfirm(char *fmt, ...); extern void msgNotify(char *fmt, ...); -extern void msgCNotify(char *fmt, ...); extern void msgWeHaveOutput(char *fmt, ...); extern int msgYesNo(char *fmt, ...); extern char *msgGetInput(char *buf, char *fmt, ...); diff --git a/usr.sbin/sysinstall/index.c b/usr.sbin/sysinstall/index.c index 6f1fc90..7a98511 100644 --- a/usr.sbin/sysinstall/index.c +++ b/usr.sbin/sysinstall/index.c @@ -4,7 +4,7 @@ * This is probably the last program in the `sysinstall' line - the next * generation being essentially a complete rewrite. * - * $Id: index.c,v 1.68 1999/05/27 10:32:45 jkh Exp $ + * $Id: index.c,v 1.69 1999/06/28 02:37:34 billf Exp $ * * Copyright (c) 1995 * Jordan Hubbard. All rights reserved. @@ -637,11 +637,19 @@ index_extract(Device *dev, PkgNodePtr top, PkgNodePtr who, Boolean depended) *cp2 = '\0'; if ((tmp2 = index_search(top, cp, NULL)) != NULL) { status = index_extract(dev, top, tmp2, TRUE); - if (DITEM_STATUS(status) != DITEM_SUCCESS) - msgCNotify("Loading of dependant package %s failed", cp); + if (DITEM_STATUS(status) != DITEM_SUCCESS) { + if (variable_get(VAR_NO_CONFIRM)) + msgNotify("Loading of dependant package %s failed", cp); + else + msgConfirm("Loading of dependant package %s failed", cp); + } + } + else if (!package_exists(cp)) { + if (variable_get(VAR_NO_CONFIRM)) + msgNotify("Warning: %s is a required package but was not found.", cp); + else + msgConfirm("Warning: %s is a required package but was not found.", cp); } - else if (!package_exists(cp)) - msgCNotify("Warning: %s is a required package but was not found.", cp); if (cp2) cp = cp2 + 1; else diff --git a/usr.sbin/sysinstall/msg.c b/usr.sbin/sysinstall/msg.c index a7116e9..7ee6d82 100644 --- a/usr.sbin/sysinstall/msg.c +++ b/usr.sbin/sysinstall/msg.c @@ -4,7 +4,7 @@ * This is probably the last program in the `sysinstall' line - the next * generation being essentially a complete rewrite. * - * $Id: msg.c,v 1.47 1997/09/20 02:48:48 jkh Exp $ + * $Id: msg.c,v 1.48 1999/05/27 10:32:48 jkh Exp $ * * Copyright (c) 1995 * Jordan Hubbard. All rights reserved. @@ -217,20 +217,6 @@ msgNotify(char *fmt, ...) dialog_msgbox(NULL, errstr, -1, -1, 0); } -/* Put up a message in a popup or notifier box, depending on NO_CONFIRM */ -void -msgCNotify(char *fmt, ...) -{ - va_list args; - - va_start(args, fmt); - if (variable_get(VAR_NO_CONFIRM)) - msgNotify(fmt, args); - else - msgConfirm(fmt, args); - va_end(args); -} - /* Put up a message in a popup yes/no box and return 1 for YES, 0 for NO */ int msgYesNo(char *fmt, ...) diff --git a/usr.sbin/sysinstall/package.c b/usr.sbin/sysinstall/package.c index 7cc980e..4c311ae 100644 --- a/usr.sbin/sysinstall/package.c +++ b/usr.sbin/sysinstall/package.c @@ -4,7 +4,7 @@ * This is probably the last program in the `sysinstall' line - the next * generation being essentially a complete rewrite. * - * $Id: package.c,v 1.76 1999/05/18 00:44:28 jkh Exp $ + * $Id: package.c,v 1.77 1999/05/27 10:32:49 jkh Exp $ * * Copyright (c) 1995 * Jordan Hubbard. All rights reserved. @@ -218,8 +218,12 @@ package_extract(Device *dev, char *name, Boolean depended) i = waitpid(pid, &tot, 0); if (sigpipe_caught || i < 0 || WEXITSTATUS(tot)) { ret = DITEM_FAILURE | DITEM_RESTORE; - msgCNotify("Add of package %s aborted, error code %d -\n" - "Please check the debug screen for more info.", name, WEXITSTATUS(tot)); + if (variable_get(VAR_NO_CONFIRM)) + msgNotify("Add of package %s aborted, error code %d -\n" + "Please check the debug screen for more info.", name, WEXITSTATUS(tot)); + else + msgConfirm("Add of package %s aborted, error code %d -\n" + "Please check the debug screen for more info.", name, WEXITSTATUS(tot)); } else msgNotify("Package %s was added successfully", name); @@ -233,8 +237,12 @@ package_extract(Device *dev, char *name, Boolean depended) } else { dialog_clear_norefresh(); - msgCNotify("Unable to fetch package %s from selected media.\n" - "No package add will be done.", name); + if (variable_get(VAR_NO_CONFIRM)) + msgNotify("Unable to fetch package %s from selected media.\n" + "No package add will be done.", name); + else + msgConfirm("Unable to fetch package %s from selected media.\n" + "No package add will be done.", name); ret = DITEM_FAILURE | DITEM_RESTORE; } signal(SIGPIPE, SIG_IGN); diff --git a/usr.sbin/sysinstall/sysinstall.h b/usr.sbin/sysinstall/sysinstall.h index c2ad3518..4816712 100644 --- a/usr.sbin/sysinstall/sysinstall.h +++ b/usr.sbin/sysinstall/sysinstall.h @@ -4,7 +4,7 @@ * This is probably the last attempt in the `sysinstall' line, the next * generation being slated to essentially a complete rewrite. * - * $Id: sysinstall.h,v 1.165 1999/05/15 14:34:22 jkh Exp $ + * $Id: sysinstall.h,v 1.166 1999/05/27 10:32:50 jkh Exp $ * * Copyright (c) 1995 * Jordan Hubbard. All rights reserved. @@ -645,7 +645,6 @@ extern void msgError(char *fmt, ...); extern void msgFatal(char *fmt, ...); extern void msgConfirm(char *fmt, ...); extern void msgNotify(char *fmt, ...); -extern void msgCNotify(char *fmt, ...); extern void msgWeHaveOutput(char *fmt, ...); extern int msgYesNo(char *fmt, ...); extern char *msgGetInput(char *buf, char *fmt, ...); |