From 157db50df72c1bcb71689312ea1760668124d9b2 Mon Sep 17 00:00:00 2001 From: Roy Marples Date: Thu, 3 Jul 2008 13:30:17 +0000 Subject: Fix is_older_than a little better, and make is_newer_than correct. --- src/rc/rc-applets.c | 21 ++++++++++++--------- 1 file changed, 12 insertions(+), 9 deletions(-) (limited to 'src/rc') diff --git a/src/rc/rc-applets.c b/src/rc/rc-applets.c index 77790e90..6a6216c4 100644 --- a/src/rc/rc-applets.c +++ b/src/rc/rc-applets.c @@ -408,7 +408,6 @@ static int do_shell_var(int argc, char **argv) void run_applets(int argc, char **argv) { int i = 2; - bool (*is_than) (const char *, const char *); char *p; pid_t pid = 0; @@ -439,20 +438,24 @@ void run_applets(int argc, char **argv) /* This test is perverted - historically the baselayout function * returns 0 on *failure*, which is plain wrong */ - if (strcmp(applet, "is_newer_than") == 0 || - strcmp(applet, "is_older_than") == 0) - { + if (strcmp(applet, "is_older_than") == 0) { if (argc < 3) exit (EXIT_FAILURE); - if (strcmp(applet, "is_newer_than") == 0) - is_than = &rc_older_than; - else - is_than = &rc_newer_than; while (i < argc) { - if (!is_than(argv[1], argv[i++])) + if (!rc_newer_than(argv[1], argv[i++])) exit(EXIT_SUCCESS); } exit(EXIT_FAILURE); + }; + /* This tets is correct as it's not present in baselayout */ + if (strcmp(applet, "is_newer_than") == 0) { + if (argc < 3) + exit (EXIT_FAILURE); + while (i < argc) { + if (!rc_newer_than(argv[1], argv[i++])) + exit(EXIT_FAILURE); + } + exit(EXIT_SUCCESS); }; if (applet[0] == 'e' || (applet[0] == 'v' && applet[1] == 'e')) -- cgit v1.2.3