the () around isspace actually matters

2005-08-16 05:39:07 +00:00 · 2005-08-16 05:39:07 +00:00 · 6ebbc7414e
commit 6ebbc7414e
parent 1eef0c4571
1 changed files with 4 additions and 2 deletions
--- a/libbb/xgetlarg.c
+++ b/libbb/xgetlarg.c
@ -21,8 +21,10 @@ extern long bb_xgetlarg(const char *arg, int base, long lower, long upper)

 	assert(arg!=NULL);

-	/* Don't allow leading whitespace. */
-	if (isspace(*arg)) {	/* Use an actual function call for minimal size. */
+	/* Don't allow leading whitespace.
+	 * Wrap isspace in () to make sure we call the 
+	 * function rather than the macro. */
+	if ((isspace)(*arg)) {
 		bb_show_usage();
 	}