[git commit] ash: remove superfluous code in arithmetic mode

Denys Vlasenko vda.linux at googlemail.com
Mon May 18 07:54:43 UTC 2015


commit: http://git.busybox.net/busybox/commit/?id=7e4ed267b645090fb576e1c876a88a9c73faabe9
branch: http://git.busybox.net/busybox/commit/?id=refs/heads/master

Based on commits 1a74845, cfc3d6a and ff13779 from
git://git.kernel.org/pub/scm/utils/dash/dash.git by Herbert Xu

function                                             old     new   delta
evalcommand                                         1197    1204      +7
localcmd                                             327     325      -2
readtoken1                                          3200    3180     -20
------------------------------------------------------------------------------
(add/remove: 0/0 grow/shrink: 1/2 up/down: 7/-22)             Total: -15 bytes

Signed-off-by: Ron Yorston <rmy at pobox.com>
Signed-off-by: Denys Vlasenko <vda.linux at googlemail.com>
---
 shell/ash.c |    7 ++-----
 1 files changed, 2 insertions(+), 5 deletions(-)

diff --git a/shell/ash.c b/shell/ash.c
index d87166c..ee7642a 100644
--- a/shell/ash.c
+++ b/shell/ash.c
@@ -11282,9 +11282,7 @@ readtoken1(int c, int syntax, char *eofmark, int striptabs)
 			goto quotemark;
 		case CENDQUOTE:
 			IF_ASH_BASH_COMPAT(bash_dollar_squote = 0;)
-			if (eofmark != NULL && arinest == 0
-			 && varnest == 0
-			) {
+			if (eofmark != NULL && varnest == 0) {
 				USTPUTC(c, out);
 			} else {
 				if (dqvarnest == 0) {
@@ -11320,7 +11318,6 @@ readtoken1(int c, int syntax, char *eofmark, int striptabs)
 				if (pgetc() == ')') {
 					if (--arinest == 0) {
 						syntax = prevsyntax;
-						dblquote = (syntax == DQSYNTAX);
 						c = CTLENDARI;
 					}
 				} else {
@@ -11645,7 +11642,7 @@ parsesub: {
 		((unsigned char *)stackblock())[typeloc] = subtype | flags;
 		if (subtype != VSNORMAL) {
 			varnest++;
-			if (dblquote || arinest) {
+			if (dblquote) {
 				dqvarnest++;
 			}
 		}


More information about the busybox-cvs mailing list