Replace space and tab checks with isblank

These are various places I found that checked for conditions equivalent
to isblank.

Signed-off-by: Jason Hobbs <jason.hobbs@calxeda.com>

diff --git a/board/hymod/env.c b/board/hymod/env.c
index c0e2cd5d6816d86ec01c9f0c5ec345a235e21291..fde428d97991271b243d6bee4fdef3efcf2e3ae9 100644 (file)
--- a/board/hymod/env.c
+++ b/board/hymod/env.c
@@ -22,6 +22,7 @@
  */
 
 #include <common.h>
+#include <linux/ctype.h>
 
 DECLARE_GLOBAL_DATA_PTR;
 
@@ -45,7 +46,7 @@ env_callback (uchar *name, uchar *value)
                nn++;
        }
 
-       while (*nn == ' ' || *nn == '\t')
+       while (isblank(*nn))
                nn++;
 
        if ((nnl = strlen (nn)) == 0) {
@@ -61,7 +62,7 @@ env_callback (uchar *name, uchar *value)
                nn[--nnl] = '\0';
        }
 
-       while (nnl > 0 && ((c = nn[nnl - 1]) == ' ' || c == '\t'))
+       while (nnl > 0 && isblank(nn[nnl - 1]))
                nn[--nnl] = '\0';
        if (nnl == 0) {
                printf ("Empty name in global env file\n");
@@ -71,11 +72,11 @@ env_callback (uchar *name, uchar *value)
        p = (char *)value;
        q = nv;
 
-       while ((c = *p) == ' ' || c == '\t')
+       while (isblank(*p))
                p++;
 
        nvl = strlen (p);
-       while (nvl > 0 && ((c = p[nvl - 1]) == ' ' || c == '\t'))
+       while (nvl > 0 && isblank(p[nvl - 1]))
                p[--nvl] = '\0';
 
        while ((*q = *p++) != '\0') {

diff --git a/common/command.c b/common/command.c
index ed931d748c095a1cd89ff0f702e10dccb72b1611..2c0bf534492e2eaaa967a50079fac5fcd18483f5 100644 (file)
--- a/common/command.c
+++ b/common/command.c
@@ -27,6 +27,7 @@
 
 #include <common.h>
 #include <command.h>
+#include <linux/ctype.h>
 
 /*
  * Use puts() instead of printf() to avoid printf buffer overflow
@@ -165,7 +166,7 @@ int var_complete(int argc, char * const argv[], char last_char, int maxv, char *
        static char tmp_buf[512];
        int space;
 
-       space = last_char == '\0' || last_char == ' ' || last_char == '\t';
+       space = last_char == '\0' || isblank(last_char);
 
        if (space && argc == 1)
                return env_complete("", maxv, cmdv, sizeof(tmp_buf), tmp_buf);
@@ -206,7 +207,7 @@ static int complete_cmdv(int argc, char * const argv[], char last_char, int maxv
        }
 
        /* more than one arg or one but the start of the next */
-       if (argc > 1 || (last_char == '\0' || last_char == ' ' || last_char == '\t')) {
+       if (argc > 1 || (last_char == '\0' || isblank(last_char))) {
                cmdtp = find_cmd(argv[0]);
                if (cmdtp == NULL || cmdtp->complete == NULL) {
                        cmdv[0] = NULL;
@@ -257,7 +258,7 @@ static int make_argv(char *s, int argvsz, char *argv[])
        while (argc < argvsz - 1) {
 
                /* skip any white space */
-               while ((*s == ' ') || (*s == '\t'))
+               while (isblank(*s))
                        ++s;
 
                if (*s == '\0') /* end of s, no more args       */
@@ -266,7 +267,7 @@ static int make_argv(char *s, int argvsz, char *argv[])
                argv[argc++] = s;       /* begin of argument string     */
 
                /* find end of string */
-               while (*s && (*s != ' ') && (*s != '\t'))
+               while (*s && !isblank(*s))
                        ++s;
 
                if (*s == '\0')         /* end of s, no more args       */

diff --git a/common/main.c b/common/main.c
index d812aa186cf3a1eba1b2a1cb89839ccc45885c66..7be295528f369122068c5f1e2ea6bfdeab6e3519 100644 (file)
--- a/common/main.c
+++ b/common/main.c
@@ -40,6 +40,7 @@
 #endif
 
 #include <post.h>
+#include <linux/ctype.h>
 
 #if defined(CONFIG_SILENT_CONSOLE) || defined(CONFIG_POST) || defined(CONFIG_CMDLINE_EDITING)
 DECLARE_GLOBAL_DATA_PTR;
@@ -1097,7 +1098,7 @@ int parse_line (char *line, char *argv[])
        while (nargs < CONFIG_SYS_MAXARGS) {
 
                /* skip any white space */
-               while ((*line == ' ') || (*line == '\t'))
+               while (isblank(*line))
                        ++line;
 
                if (*line == '\0') {    /* end of line, no more args    */
@@ -1111,7 +1112,7 @@ int parse_line (char *line, char *argv[])
                argv[nargs++] = line;   /* begin of argument string     */
 
                /* find end of string */
-               while (*line && (*line != ' ') && (*line != '\t'))
+               while (*line && !isblank(*line))
                        ++line;
 
                if (*line == '\0') {    /* end of line, no more args    */

diff --git a/drivers/bios_emulator/x86emu/debug.c b/drivers/bios_emulator/x86emu/debug.c
index 241acf3a89a566190e130f787019bf8ec53bac6c..6417d099988d27b392caf8fcd9a4e9e8af4c7ca0 100644 (file)
--- a/drivers/bios_emulator/x86emu/debug.c
+++ b/drivers/bios_emulator/x86emu/debug.c
@@ -39,6 +39,7 @@
 
 #include <stdarg.h>
 #include <common.h>
+#include <linux/ctype.h>
 #include "x86emu/x86emui.h"
 
 /*----------------------------- Implementation ----------------------------*/
@@ -333,7 +334,7 @@ static int x86emu_parse_line(char *s, int *ps, int *n)
        int cmd;
 
        *n = 0;
-       while (*s == ' ' || *s == '\t')
+       while (isblank(*s))
                s++;
        ps[*n] = *s;
        switch (*s) {
@@ -346,13 +347,13 @@ static int x86emu_parse_line(char *s, int *ps, int *n)
        }
 
        while (1) {
-               while (*s != ' ' && *s != '\t' && *s != '\n')
+               while (!isblank(*s) && *s != '\n')
                        s++;
 
                if (*s == '\n')
                        return cmd;
 
-               while (*s == ' ' || *s == '\t')
+               while (isblank(*s))
                        s++;
 
                *n += 1;

diff --git a/examples/standalone/smc911x_eeprom.c b/examples/standalone/smc911x_eeprom.c
index 00e826689f31b5001d3a3846c74088fab29254a4..6c79c5f65a0f74236f46c1f0da26612d3705d435 100644 (file)
--- a/examples/standalone/smc911x_eeprom.c
+++ b/examples/standalone/smc911x_eeprom.c
@@ -16,6 +16,7 @@
 
 #include <common.h>
 #include <exports.h>
+#include <linux/ctype.h>
 #include "../drivers/net/smc911x.h"
 
 /**
@@ -128,7 +129,7 @@ static int write_eeprom_reg(struct eth_device *dev, u8 value, u8 reg)
  */
 static char *skip_space(char *buf)
 {
-       while (buf[0] == ' ' || buf[0] == '\t')
+       while (isblank(buf[0]))
                ++buf;
        return buf;
 }
@@ -357,7 +358,7 @@ int smc911x_eeprom(int argc, char * const argv[])
                        continue;
 
                /* Only accept 1 letter commands */
-               if (line[0] && line[1] && line[1] != ' ' && line[1] != '\t')
+               if (line[0] && line[1] && !isblank(line[1]))
                        goto unknown_cmd;
 
                /* Now parse the command */

diff --git a/lib/hashtable.c b/lib/hashtable.c
index 026dbca2fadecbd4533ca5d552973dc532d78c97..6895550d3cae5a7ac476f3aa8770542fefd87e1f 100644 (file)
--- a/lib/hashtable.c
+++ b/lib/hashtable.c
@@ -32,6 +32,7 @@
 #ifdef USE_HOSTCC              /* HOST build */
 # include <string.h>
 # include <assert.h>
+# include <ctype.h>
 
 # ifndef debug
 #  ifdef DEBUG
@@ -43,6 +44,7 @@
 #else                          /* U-Boot build */
 # include <common.h>
 # include <linux/string.h>
+# include <linux/ctype.h>
 #endif
 
 #ifndef        CONFIG_ENV_MIN_ENTRIES  /* minimum number of entries */
@@ -690,7 +692,7 @@ int himport_r(struct hsearch_data *htab,
                ENTRY e, *rv;
 
                /* skip leading white space */
-               while ((*dp == ' ') || (*dp == '\t'))
+               while (isblank(*dp))
                        ++dp;
 
                /* skip comment lines */