shithub: libmujs

Download patch

ref: 66cf4ccd82d4425c9c53093bc745c19a711f93a9
parent: 849fb89be7febd26a89d006b91866be3651d3a45
author: Tor Andersson <[email protected]>
date: Mon Jan 22 08:36:30 EST 2018 

Fix issue #61: Silence misleading indentation warning.

Add NULL check to pexpi to simplify call sites.


--- a/jsdump.c
+++ b/jsdump.c
@@ -276,8 +276,12 @@
 
 static void pexpi(int d, int p, js_Ast *exp)
 {
-	int tp = prec(exp->type);
-	int paren = 0;
+	int tp, paren;
+
+	if (!exp) return;
+
+	tp = prec(exp->type);
+	paren = 0;
 	if (tp < p) {
 		pc('(');
 		paren = 1;
@@ -389,7 +393,7 @@
 	case EXP_FUN:
 		if (p == 0) pc('(');
 		ps("function ");
-		if (exp->a) pexpi(d, 0, exp->a);
+		pexpi(d, 0, exp->a);
 		pc('(');
 		pargs(d, exp->b);
 		ps(") {\n");
@@ -519,16 +523,16 @@
 
 	case STM_FOR:
 		ps("for (");
-		if (stm->a) pexp(d, stm->a); ps("; ");
-		if (stm->b) pexp(d, stm->b); ps("; ");
-		if (stm->c) pexp(d, stm->c); ps(")");
+		pexp(d, stm->a); ps("; ");
+		pexp(d, stm->b); ps("; ");
+		pexp(d, stm->c); ps(")");
 		pstmh(d, stm->d);
 		break;
 	case STM_FOR_VAR:
 		ps("for (var ");
 		pvarlist(d, stm->a); ps("; ");
-		if (stm->b) pexp(d, stm->b); ps("; ");
-		if (stm->c) pexp(d, stm->c); ps(")");
+		pexp(d, stm->b); ps("; ");
+		pexp(d, stm->c); ps(")");
 		pstmh(d, stm->d);
 		break;
 	case STM_FOR_IN: