aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDenis Vlasenko <vda.linux@googlemail.com>2008-02-12 06:04:06 +0000
committerDenis Vlasenko <vda.linux@googlemail.com>2008-02-12 06:04:06 +0000
commit648b44fd785f8d23e363886ff306e8ef70d148ba (patch)
treeedff6d790ab04e2c5221b34e2a6f65409614e491
parenta5f2cd30c9308b3cec359acdf79d60958d6d5834 (diff)
downloadbusybox-648b44fd785f8d23e363886ff306e8ef70d148ba.tar.gz
msh: shrink, mainly by using smallints
function old new delta synio 251 263 +12 filechar 310 315 +5 subgetc 781 785 +4 brkcontin 106 109 +3 ... yylex 981 976 -5 execute 2513 2508 -5 waitfor 321 315 -6 onintr 103 97 -6 doset 332 326 -6 doexec 186 180 -6 yyparse 65 57 -8 thenpart 135 126 -9 forkexec 1379 1370 -9 onecommand 457 433 -24 newtp 52 28 -24 command 1086 1061 -25 ------------------------------------------------------------------------------ (add/remove: 0/0 grow/shrink: 7/23 up/down: 27/-163) Total: -136 bytes
-rw-r--r--shell/msh.c118
1 files changed, 57 insertions, 61 deletions
diff --git a/shell/msh.c b/shell/msh.c
index f05028f26..99b03947e 100644
--- a/shell/msh.c
+++ b/shell/msh.c
@@ -158,7 +158,7 @@ int mshdbg_rc = 0;
/*
* library and system definitions
*/
-typedef void xint; /* base type of jmp_buf, for not broken compilers */
+typedef void xint; /* base type of jmp_buf, for not broken compilers */
/*
* shell components
@@ -172,20 +172,20 @@ typedef void xint; /* base type of jmp_buf, for not broken compilers */
* redirection
*/
struct ioword {
- short io_unit; /* unit affected */
- short io_flag; /* action (below) */
- char *io_name; /* file name */
+ short io_unit; /* unit affected */
+ short io_flag; /* action (below) */
+ char *io_name; /* file name */
};
-#define IOREAD 1 /* < */
-#define IOHERE 2 /* << (here file) */
-#define IOWRITE 4 /* > */
-#define IOCAT 8 /* >> */
-#define IOXHERE 16 /* ${}, ` in << */
-#define IODUP 32 /* >&digit */
-#define IOCLOSE 64 /* >&- */
+#define IOREAD 1 /* < */
+#define IOHERE 2 /* << (here file) */
+#define IOWRITE 4 /* > */
+#define IOCAT 8 /* >> */
+#define IOXHERE 16 /* ${}, ` in << */
+#define IODUP 32 /* >&digit */
+#define IOCLOSE 64 /* >&- */
-#define IODEFAULT (-1) /* token for default IO unit */
+#define IODEFAULT (-1) /* token for default IO unit */
/*
@@ -193,12 +193,12 @@ struct ioword {
* Might eventually use a union.
*/
struct op {
- int type; /* operation type, see below */
- char **words; /* arguments to a command */
- struct ioword **ioact; /* IO actions (eg, < > >>) */
+ smallint type; /* operation type, see Txxxx below */
+ char **words; /* arguments to a command */
+ struct ioword **ioact; /* IO actions (eg, < > >>) */
struct op *left;
struct op *right;
- char *str; /* identifier for case and for */
+ char *str; /* identifier for case and for */
};
#define TCOM 1 /* command */
@@ -269,9 +269,9 @@ struct brkcon {
};
-static int trapset; /* trap pending */
+static smallint trapset; /* trap pending (signal number) */
-static int yynerrs; /* yacc */
+static smallint yynerrs; /* yacc (flag) */
/* moved to G: static char line[LINELIM]; */
@@ -420,7 +420,7 @@ struct ioarg {
char **awordlist;
int afile; /* file descriptor */
unsigned afid; /* buffer id */
- long afpos; /* file position */
+ off_t afpos; /* file position */
struct iobuf *afbuf; /* buffer for this file */
};
@@ -539,7 +539,7 @@ static int xxchar(struct ioarg *ap);
struct here {
char *h_tag;
- int h_dosub;
+ char h_dosub;
struct ioword *h_iop;
struct here *h_next;
};
@@ -600,15 +600,15 @@ static struct op *dowholefile(int, int);
static char **dolv;
static int dolc;
static int exstat;
-static char gflg;
-static int interactive; /* Is this an interactive shell */
-static int execflg;
-static int multiline; /* \n changed to ; */
-static struct op *outtree; /* result from parser */
+static smallint gflg; /* (seems to be a parse error indicator) */
+static smallint interactive; /* Is this an interactive shell */
+static smallint execflg;
+static smallint isbreak; /* "break" statement was seen */
+static int multiline; /* '\n' changed to ';' (counter) */
+static struct op *outtree; /* result from parser */
static xint *failpt;
static xint *errpt;
static struct brkcon *brklist;
-static int isbreak;
static struct wdblock *wdlist;
static struct wdblock *iolist;
@@ -624,10 +624,10 @@ static struct var *shell; /* shell to interpret command files */
static struct var *ifs; /* field separators */
static int areanum; /* current allocation area */
-static int intr; /* interrupt pending */
+static smallint intr; /* interrupt pending (bool) */
+static smallint heedint = 1; /* heed interrupt signals (bool) */
static int inparse;
static char *null = (char*)""; /* null value for variable */
-static int heedint = 1; /* heed interrupt signals */
static void (*qflag)(int) = SIG_IGN;
static int startl;
static int peeksym;
@@ -1560,10 +1560,11 @@ static int gmatch(const char *s, const char *p)
static void yyerror(const char *s) ATTRIBUTE_NORETURN;
static void yyerror(const char *s)
{
- yynerrs++;
+ yynerrs = 1;
if (interactive && global_env.iop <= iostack) {
multiline = 0;
- while (eofc() == 0 && yylex(0) != '\n');
+ while (eofc() == 0 && yylex(0) != '\n')
+ continue;
}
err(s);
fail();
@@ -1584,7 +1585,7 @@ int yyparse(void)
yynerrs = 0;
outtree = c_list();
musthave('\n', 0);
- return (yynerrs != 0);
+ return yynerrs; /* 0/1 */
}
static struct op *pipeline(int cf)
@@ -1819,7 +1820,7 @@ static struct op *command(int cf)
case UNTIL:
multiline++;
t = newtp();
- t->type = c == WHILE ? TWHILE : TUNTIL;
+ t->type = (c == WHILE ? TWHILE : TUNTIL);
t->left = c_list();
t->right = dogroup(1);
t->words = NULL;
@@ -1916,7 +1917,7 @@ static struct op *thenpart(void)
return NULL;
}
t = newtp();
- t->type = 0;
+ /*t->type = 0; - newtp() did this */
t->left = c_list();
if (t->left == NULL)
zzerr();
@@ -2044,8 +2045,7 @@ static struct op *block(int type, struct op *t1, struct op *t2, char **wp)
t->right = t2;
t->words = wp;
- DBGPRINTF7(("BLOCK: inserted %p between %p and %p\n", t, t1,
- t2));
+ DBGPRINTF7(("BLOCK: inserted %p between %p and %p\n", t, t1, t2));
return t;
}
@@ -2099,12 +2099,7 @@ static struct op *newtp(void)
struct op *t;
t = (struct op *) tree(sizeof(*t));
- t->type = 0;
- t->words = NULL;
- t->ioact = NULL;
- t->left = NULL;
- t->right = NULL;
- t->str = NULL;
+ memset(t, 0, sizeof(*t));
DBGPRINTF3(("NEWTP: allocated %p\n", t));
@@ -2481,7 +2476,8 @@ static int execute(struct op *t, int *pin, int *pout, int act)
rv = 0;
a = areanum++;
- wp = (wp2 = t->words) != NULL
+ wp2 = t->words;
+ wp = (wp2 != NULL)
? eval(wp2, t->type == TCOM ? DOALL : DOALL & ~DOKEY)
: NULL;
@@ -2533,7 +2529,7 @@ static int execute(struct op *t, int *pin, int *pout, int act)
case TASYNC:
{
- int hinteractive = interactive;
+ smallint hinteractive = interactive;
DBGPRINTF7(("EXECUTE: TASYNC clause, calling vfork()...\n"));
@@ -2700,10 +2696,10 @@ static int forkexec(struct op *t, int *pin, int *pout, int act, char **wp)
int *hpin = pin;
int *hpout = pout;
char *hwp;
- int hinteractive;
- int hintr;
+ smallint hinteractive;
+ smallint hintr;
+ smallint hexecflg;
struct brkcon *hbrklist;
- int hexecflg;
#if __GNUC__
/* Avoid longjmp clobbering */
@@ -2993,7 +2989,7 @@ static int waitfor(int lastpid, int canintr)
{
int pid, rv;
int s;
- int oheedint = heedint;
+ smallint oheedint = heedint;
heedint = 0;
rv = 0;
@@ -3521,7 +3517,7 @@ static int brkcontin(char *cp, int val)
err("bad break/continue level");
return 1;
}
- isbreak = val;
+ isbreak = (val != 0);
longjmp(bc->brkpt, 1);
/* NOTREACHED */
}
@@ -3699,7 +3695,7 @@ static char **eval(char **ap, int f)
expand(*wf, &wb, f & ~DOGLOB);
}
}
- for (wb = addword((char *) 0, wb); *ap; ap++) {
+ for (wb = addword((char *) NULL, wb); *ap; ap++) {
if (!FLAG['k'] || !isassign(*ap))
expand(*ap, &wb, f & ~DOKEY);
}
@@ -3935,13 +3931,13 @@ static int dollar(int quoted)
global_env.iop->task = otask;
if (c != '}') {
err("unclosed ${");
- gflg++;
+ gflg = 1;
return c;
}
}
if (global_env.linep >= elinep) {
err("string in ${} too long");
- gflg++;
+ gflg = 1;
global_env.linep -= 10;
}
*global_env.linep = 0;
@@ -3971,7 +3967,7 @@ static int dollar(int quoted)
case '=':
if (isdigit(*s)) {
err("cannot use ${...=...} with $n");
- gflg++;
+ gflg = 1;
break;
}
setval(vp, cp);
@@ -3988,7 +3984,7 @@ static int dollar(int quoted)
err(s);
} else
err(cp);
- gflg++;
+ gflg = 1;
break;
}
} else if (c == '+')
@@ -3996,7 +3992,7 @@ static int dollar(int quoted)
if (FLAG['u'] && dolp == null) {
prs("unset variable: ");
err(s);
- gflg++;
+ gflg = 1;
}
global_env.linep = s;
PUSHIO(aword, dolp, quoted ? qstrchar : strchar);
@@ -4539,7 +4535,7 @@ static int my_getc(int ec)
if (global_env.linep > elinep) {
while ((c = readc()) != '\n' && c);
err("input line too long");
- gflg++;
+ gflg = 1;
return c;
}
c = readc();
@@ -4638,7 +4634,7 @@ static void pushio(struct ioarg *argp, int (*fn) (struct ioarg *))
if (++global_env.iop >= &iostack[NPUSH]) {
global_env.iop--;
err("Shell input nested too deeply");
- gflg++;
+ gflg = 1;
return;
}
@@ -5012,7 +5008,7 @@ static void markhere(char *s, struct ioword *iop)
*s &= ~QUOTE;
}
}
- h->h_dosub = iop->io_flag & IOXHERE;
+ h->h_dosub = ((iop->io_flag & IOXHERE) ? '\0' : '\'');
}
static void gethere(void)
@@ -5023,7 +5019,7 @@ static void gethere(void)
/* Scan here files first leaving inhere list in place */
for (hp = h = inhere; h != NULL; hp = h, h = h->h_next)
- readhere(&h->h_iop->io_name, h->h_tag, h->h_dosub ? 0 : '\'');
+ readhere(&h->h_iop->io_name, h->h_tag, h->h_dosub /* NUL or ' */);
/* Make inhere list active - keep list intact for scraphere */
if (hp != NULL) {
@@ -5288,7 +5284,7 @@ int msh_main(int argc, char **argv)
break;
case 'i':
- interactive++;
+ interactive = 1;
default:
if (*s >= 'a' && *s <= 'z')
FLAG[(int) *s]++;
@@ -5324,7 +5320,7 @@ int msh_main(int argc, char **argv)
if (global_env.iop < iostack) {
PUSHIO(afile, 0, iof);
if (isatty(0) && isatty(1) && !cflag) {
- interactive++;
+ interactive = 1;
#if !ENABLE_FEATURE_SH_EXTRA_QUIET
#ifdef MSHDEBUG
printf("\n\n%s built-in shell (msh with debug)\n", bb_banner);
@@ -5338,7 +5334,7 @@ int msh_main(int argc, char **argv)
signal(SIGQUIT, qflag);
if (name && name[0] == '-') {
- interactive++;
+ interactive = 1;
f = open(".profile", O_RDONLY);
if (f >= 0)
next(remap(f));