From b134da55059775759b6e2d1ab8d6425033c3834a Mon Sep 17 00:00:00 2001 From: Paul Smith Date: Wed, 25 Aug 1999 21:39:28 +0000 Subject: [PATCH] * A few cleanups, and 3.77.94 release. --- ChangeLog | 19 +++++++++++++++++-- function.c | 9 +++------ main.c | 23 +++++++++++++++-------- make.h | 11 +++++++++++ variable.c | 3 +-- 5 files changed, 47 insertions(+), 18 deletions(-) diff --git a/ChangeLog b/ChangeLog index bac8e47..c273f09 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,18 @@ +1999-08-25 Paul D. Smith + + * Version 3.77.94 released. + + * main.c (main) [__MSDOS__]: If the user uses -j, warn that it's + not supported and reset it. + + * make.h (ISDIGIT): Obtained this from the textutils distribution. + * main.c (decode_switches): Use it. + * function.c (is_numeric): Use it. + + * main.c (struct command_switch): Store the switch char in an + unsigned char to shut up GCC about using it with ctype.h macros. + Besides, it _is_ always unsigned. + 1999-08-24 Paul D. Smith * make.texinfo: Change "dependency" to "prerequisite" and @@ -13,8 +28,8 @@ * remake.c (update_file): Move the considered check into the double-colon rule loop, so we consider double-colon rules - individually (otherwise after the first is pruned, the rest might - not get run). + individually (otherwise after the first is pruned, the rest won't + get run). * README.template: Minor changes. diff --git a/function.c b/function.c index 0e619fd..0acfd7e 100644 --- a/function.c +++ b/function.c @@ -702,13 +702,10 @@ is_numeric (p) char *end = p + strlen (p) - 1; char *beg = p; strip_whitespace (&p, &end); - while (p <= end) - { - if (!isdigit (*p)) - return 0; - p++; - } + while (p <= end) + if (!ISDIGIT (*(p++))) /* ISDIGIT only evals its arg once: see make.h. */ + return 0; return (end - beg >= 0); } diff --git a/main.c b/main.c index de82b59..0035a1d 100644 --- a/main.c +++ b/main.c @@ -79,7 +79,7 @@ static char *quote_as_word PARAMS ((char *out, char *in, int double_dollars)); struct command_switch { - char c; /* The switch character. */ + unsigned char c; /* The switch character. */ enum /* Type of the value. */ { @@ -1270,7 +1270,7 @@ int main (int argc, char ** argv) /* We need to know what kind of shell we will be using. */ { extern int _is_unixy_shell (const char *_path); - struct variable *shv = lookup_variable("SHELL", 5); + struct variable *shv = lookup_variable ("SHELL", 5); extern int unixy_shell; extern char *default_shell; @@ -1294,6 +1294,15 @@ int main (int argc, char ** argv) decode_env_switches ("MFLAGS", 6); #endif +#ifdef __MSDOS__ + if (job_slots != 1) + { + error (NILF, _("Parallel jobs (-j) are not supported on MS-DOS.")); + error (NILF, _("Resetting to single job mode.")); + job_slots = 1; + } +#endif + #ifdef MAKE_JOBSERVER /* If the jobserver-fds option is seen, make sure that -j is reasonable. */ @@ -1851,8 +1860,7 @@ handle_non_switch_argument (arg, env) } else { - lastgoal->next - = (struct dep *) xmalloc (sizeof (struct dep)); + lastgoal->next = (struct dep *) xmalloc (sizeof (struct dep)); lastgoal = lastgoal->next; } lastgoal->name = 0; @@ -2075,7 +2083,7 @@ decode_switches (argc, argv, env) case positive_int: if (optarg == 0 && argc > optind - && isdigit (argv[optind][0])) + && ISDIGIT (argv[optind][0])) optarg = argv[optind++]; if (!doit) @@ -2087,8 +2095,7 @@ decode_switches (argc, argv, env) if (i < 1) { if (doit) - error (NILF, _("the `-%c' option requires a \ -positive integral argument"), + error (NILF, _("the `-%c' option requires a positive integral argument"), cs->c); bad = 1; } @@ -2103,7 +2110,7 @@ positive integral argument"), #ifndef NO_FLOAT case floating: if (optarg == 0 && optind < argc - && (isdigit (argv[optind][0]) || argv[optind][0] == '.')) + && (ISDIGIT (argv[optind][0]) || argv[optind][0] == '.')) optarg = argv[optind++]; if (doit) diff --git a/make.h b/make.h index b25318b..a50334f 100644 --- a/make.h +++ b/make.h @@ -299,6 +299,17 @@ extern char *alloca (); # endif /* HAVE_ALLOCA_H. */ #endif /* GCC. */ +/* ISDIGIT offers the following features: + - Its arg may be any int or unsigned int; it need not be an unsigned char. + - It's guaranteed to evaluate its argument exactly once. + NOTE! Make relies on this behavior, don't change it! + - It's typically faster. + Posix 1003.2-1992 section 2.5.2.1 page 50 lines 1556-1558 says that + only '0' through '9' are digits. Prefer ISDIGIT to isdigit() unless + it's important to use the locale's definition of `digit' even when the + host does not conform to Posix. */ +#define ISDIGIT(c) ((unsigned) (c) - '0' <= 9) + #ifndef iAPX286 # define streq(a, b) \ ((a) == (b) || \ diff --git a/variable.c b/variable.c index 97207f1..377daf5 100644 --- a/variable.c +++ b/variable.c @@ -954,7 +954,7 @@ print_variable (v, prefix) register struct variable *v; char *prefix; { - char *origin; + const char *origin; switch (v->origin) { @@ -982,7 +982,6 @@ print_variable (v, prefix) case o_invalid: default: abort (); - break; } printf ("# %s\n", origin); -- 2.7.4