[git commit] bc: trivial simplification in bc_parse_return()

Denys Vlasenko vda.linux at googlemail.com
Wed Dec 5 19:28:26 UTC 2018


commit: https://git.busybox.net/busybox/commit/?id=452df923f7f5ae47f3076d7da562946af6123ca9
branch: https://git.busybox.net/busybox/commit/?id=refs/heads/master

function                                             old     new   delta
bc_parse_stmt                                       2276    2271      -5

Signed-off-by: Denys Vlasenko <vda.linux at googlemail.com>
---
 miscutils/bc.c | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/miscutils/bc.c b/miscutils/bc.c
index 656ecd892..b454d4b2b 100644
--- a/miscutils/bc.c
+++ b/miscutils/bc.c
@@ -4047,14 +4047,11 @@ static BcStatus bc_parse_return(BcParse *p)
 	else {
 
 		s = bc_parse_expr(p, 0, bc_parse_next_expr);
-		if (s && s != BC_STATUS_PARSE_EMPTY_EXP)
-			return s;
-
 		if (s == BC_STATUS_PARSE_EMPTY_EXP) {
 			bc_parse_push(p, BC_INST_RET0);
 			s = bc_lex_next(&p->l);
-			if (s) return s;
 		}
+		if (s) return s;
 
 		if (!paren || p->l.t.last != BC_LEX_RPAREN) {
 			s = bc_posix_error("POSIX requires parentheses around return expressions");


More information about the busybox-cvs mailing list