mirror of
https://github.com/u-boot/u-boot.git
synced 2025-04-26 07:17:10 +00:00
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>
This commit is contained in:
parent
ce2d4c9532
commit
4d91a6ecab
6 changed files with 23 additions and 16 deletions
|
@ -22,6 +22,7 @@
|
||||||
*/
|
*/
|
||||||
|
|
||||||
#include <common.h>
|
#include <common.h>
|
||||||
|
#include <linux/ctype.h>
|
||||||
|
|
||||||
DECLARE_GLOBAL_DATA_PTR;
|
DECLARE_GLOBAL_DATA_PTR;
|
||||||
|
|
||||||
|
@ -45,7 +46,7 @@ env_callback (uchar *name, uchar *value)
|
||||||
nn++;
|
nn++;
|
||||||
}
|
}
|
||||||
|
|
||||||
while (*nn == ' ' || *nn == '\t')
|
while (isblank(*nn))
|
||||||
nn++;
|
nn++;
|
||||||
|
|
||||||
if ((nnl = strlen (nn)) == 0) {
|
if ((nnl = strlen (nn)) == 0) {
|
||||||
|
@ -61,7 +62,7 @@ env_callback (uchar *name, uchar *value)
|
||||||
nn[--nnl] = '\0';
|
nn[--nnl] = '\0';
|
||||||
}
|
}
|
||||||
|
|
||||||
while (nnl > 0 && ((c = nn[nnl - 1]) == ' ' || c == '\t'))
|
while (nnl > 0 && isblank(nn[nnl - 1]))
|
||||||
nn[--nnl] = '\0';
|
nn[--nnl] = '\0';
|
||||||
if (nnl == 0) {
|
if (nnl == 0) {
|
||||||
printf ("Empty name in global env file\n");
|
printf ("Empty name in global env file\n");
|
||||||
|
@ -71,11 +72,11 @@ env_callback (uchar *name, uchar *value)
|
||||||
p = (char *)value;
|
p = (char *)value;
|
||||||
q = nv;
|
q = nv;
|
||||||
|
|
||||||
while ((c = *p) == ' ' || c == '\t')
|
while (isblank(*p))
|
||||||
p++;
|
p++;
|
||||||
|
|
||||||
nvl = strlen (p);
|
nvl = strlen (p);
|
||||||
while (nvl > 0 && ((c = p[nvl - 1]) == ' ' || c == '\t'))
|
while (nvl > 0 && isblank(p[nvl - 1]))
|
||||||
p[--nvl] = '\0';
|
p[--nvl] = '\0';
|
||||||
|
|
||||||
while ((*q = *p++) != '\0') {
|
while ((*q = *p++) != '\0') {
|
||||||
|
|
|
@ -27,6 +27,7 @@
|
||||||
|
|
||||||
#include <common.h>
|
#include <common.h>
|
||||||
#include <command.h>
|
#include <command.h>
|
||||||
|
#include <linux/ctype.h>
|
||||||
|
|
||||||
/*
|
/*
|
||||||
* Use puts() instead of printf() to avoid printf buffer overflow
|
* 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];
|
static char tmp_buf[512];
|
||||||
int space;
|
int space;
|
||||||
|
|
||||||
space = last_char == '\0' || last_char == ' ' || last_char == '\t';
|
space = last_char == '\0' || isblank(last_char);
|
||||||
|
|
||||||
if (space && argc == 1)
|
if (space && argc == 1)
|
||||||
return env_complete("", maxv, cmdv, sizeof(tmp_buf), tmp_buf);
|
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 */
|
/* 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]);
|
cmdtp = find_cmd(argv[0]);
|
||||||
if (cmdtp == NULL || cmdtp->complete == NULL) {
|
if (cmdtp == NULL || cmdtp->complete == NULL) {
|
||||||
cmdv[0] = NULL;
|
cmdv[0] = NULL;
|
||||||
|
@ -257,7 +258,7 @@ static int make_argv(char *s, int argvsz, char *argv[])
|
||||||
while (argc < argvsz - 1) {
|
while (argc < argvsz - 1) {
|
||||||
|
|
||||||
/* skip any white space */
|
/* skip any white space */
|
||||||
while ((*s == ' ') || (*s == '\t'))
|
while (isblank(*s))
|
||||||
++s;
|
++s;
|
||||||
|
|
||||||
if (*s == '\0') /* end of s, no more args */
|
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 */
|
argv[argc++] = s; /* begin of argument string */
|
||||||
|
|
||||||
/* find end of string */
|
/* find end of string */
|
||||||
while (*s && (*s != ' ') && (*s != '\t'))
|
while (*s && !isblank(*s))
|
||||||
++s;
|
++s;
|
||||||
|
|
||||||
if (*s == '\0') /* end of s, no more args */
|
if (*s == '\0') /* end of s, no more args */
|
||||||
|
|
|
@ -40,6 +40,7 @@
|
||||||
#endif
|
#endif
|
||||||
|
|
||||||
#include <post.h>
|
#include <post.h>
|
||||||
|
#include <linux/ctype.h>
|
||||||
|
|
||||||
#if defined(CONFIG_SILENT_CONSOLE) || defined(CONFIG_POST) || defined(CONFIG_CMDLINE_EDITING)
|
#if defined(CONFIG_SILENT_CONSOLE) || defined(CONFIG_POST) || defined(CONFIG_CMDLINE_EDITING)
|
||||||
DECLARE_GLOBAL_DATA_PTR;
|
DECLARE_GLOBAL_DATA_PTR;
|
||||||
|
@ -1097,7 +1098,7 @@ int parse_line (char *line, char *argv[])
|
||||||
while (nargs < CONFIG_SYS_MAXARGS) {
|
while (nargs < CONFIG_SYS_MAXARGS) {
|
||||||
|
|
||||||
/* skip any white space */
|
/* skip any white space */
|
||||||
while ((*line == ' ') || (*line == '\t'))
|
while (isblank(*line))
|
||||||
++line;
|
++line;
|
||||||
|
|
||||||
if (*line == '\0') { /* end of line, no more args */
|
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 */
|
argv[nargs++] = line; /* begin of argument string */
|
||||||
|
|
||||||
/* find end of string */
|
/* find end of string */
|
||||||
while (*line && (*line != ' ') && (*line != '\t'))
|
while (*line && !isblank(*line))
|
||||||
++line;
|
++line;
|
||||||
|
|
||||||
if (*line == '\0') { /* end of line, no more args */
|
if (*line == '\0') { /* end of line, no more args */
|
||||||
|
|
|
@ -39,6 +39,7 @@
|
||||||
|
|
||||||
#include <stdarg.h>
|
#include <stdarg.h>
|
||||||
#include <common.h>
|
#include <common.h>
|
||||||
|
#include <linux/ctype.h>
|
||||||
#include "x86emu/x86emui.h"
|
#include "x86emu/x86emui.h"
|
||||||
|
|
||||||
/*----------------------------- Implementation ----------------------------*/
|
/*----------------------------- Implementation ----------------------------*/
|
||||||
|
@ -333,7 +334,7 @@ static int x86emu_parse_line(char *s, int *ps, int *n)
|
||||||
int cmd;
|
int cmd;
|
||||||
|
|
||||||
*n = 0;
|
*n = 0;
|
||||||
while (*s == ' ' || *s == '\t')
|
while (isblank(*s))
|
||||||
s++;
|
s++;
|
||||||
ps[*n] = *s;
|
ps[*n] = *s;
|
||||||
switch (*s) {
|
switch (*s) {
|
||||||
|
@ -346,13 +347,13 @@ static int x86emu_parse_line(char *s, int *ps, int *n)
|
||||||
}
|
}
|
||||||
|
|
||||||
while (1) {
|
while (1) {
|
||||||
while (*s != ' ' && *s != '\t' && *s != '\n')
|
while (!isblank(*s) && *s != '\n')
|
||||||
s++;
|
s++;
|
||||||
|
|
||||||
if (*s == '\n')
|
if (*s == '\n')
|
||||||
return cmd;
|
return cmd;
|
||||||
|
|
||||||
while (*s == ' ' || *s == '\t')
|
while (isblank(*s))
|
||||||
s++;
|
s++;
|
||||||
|
|
||||||
*n += 1;
|
*n += 1;
|
||||||
|
|
|
@ -16,6 +16,7 @@
|
||||||
|
|
||||||
#include <common.h>
|
#include <common.h>
|
||||||
#include <exports.h>
|
#include <exports.h>
|
||||||
|
#include <linux/ctype.h>
|
||||||
#include "../drivers/net/smc911x.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)
|
static char *skip_space(char *buf)
|
||||||
{
|
{
|
||||||
while (buf[0] == ' ' || buf[0] == '\t')
|
while (isblank(buf[0]))
|
||||||
++buf;
|
++buf;
|
||||||
return buf;
|
return buf;
|
||||||
}
|
}
|
||||||
|
@ -357,7 +358,7 @@ int smc911x_eeprom(int argc, char * const argv[])
|
||||||
continue;
|
continue;
|
||||||
|
|
||||||
/* Only accept 1 letter commands */
|
/* 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;
|
goto unknown_cmd;
|
||||||
|
|
||||||
/* Now parse the command */
|
/* Now parse the command */
|
||||||
|
|
|
@ -32,6 +32,7 @@
|
||||||
#ifdef USE_HOSTCC /* HOST build */
|
#ifdef USE_HOSTCC /* HOST build */
|
||||||
# include <string.h>
|
# include <string.h>
|
||||||
# include <assert.h>
|
# include <assert.h>
|
||||||
|
# include <ctype.h>
|
||||||
|
|
||||||
# ifndef debug
|
# ifndef debug
|
||||||
# ifdef DEBUG
|
# ifdef DEBUG
|
||||||
|
@ -43,6 +44,7 @@
|
||||||
#else /* U-Boot build */
|
#else /* U-Boot build */
|
||||||
# include <common.h>
|
# include <common.h>
|
||||||
# include <linux/string.h>
|
# include <linux/string.h>
|
||||||
|
# include <linux/ctype.h>
|
||||||
#endif
|
#endif
|
||||||
|
|
||||||
#ifndef CONFIG_ENV_MIN_ENTRIES /* minimum number of entries */
|
#ifndef CONFIG_ENV_MIN_ENTRIES /* minimum number of entries */
|
||||||
|
@ -690,7 +692,7 @@ int himport_r(struct hsearch_data *htab,
|
||||||
ENTRY e, *rv;
|
ENTRY e, *rv;
|
||||||
|
|
||||||
/* skip leading white space */
|
/* skip leading white space */
|
||||||
while ((*dp == ' ') || (*dp == '\t'))
|
while (isblank(*dp))
|
||||||
++dp;
|
++dp;
|
||||||
|
|
||||||
/* skip comment lines */
|
/* skip comment lines */
|
||||||
|
|
Loading…
Add table
Reference in a new issue