fix bugs: multibyte chars, null pointer in a list

node/expr/array.go

@@ -26,7 +26,9 @@ func (n *Array) Walk(v node.Visitor) {
 	if n.Items != nil {
 		vv := v.GetChildrenVisitor("Items")
 		for _, nn := range n.Items {
-			nn.Walk(vv)
+			if nn != nil {
+				nn.Walk(vv)
+			}
 		}
 	}
 

node/expr/closure.go

@@ -42,14 +42,18 @@ func (n *Closure) Walk(v node.Visitor) {
 	if n.Params != nil {
 		vv := v.GetChildrenVisitor("Params")
 		for _, nn := range n.Params {
-			nn.Walk(vv)
+			if nn != nil {
+				nn.Walk(vv)
+			}
 		}
 	}
 
 	if n.Uses != nil {
 		vv := v.GetChildrenVisitor("Uses")
 		for _, nn := range n.Uses {
-			nn.Walk(vv)
+			if nn != nil {
+				nn.Walk(vv)
+			}
 		}
 	}
 
@@ -61,7 +65,9 @@ func (n *Closure) Walk(v node.Visitor) {
 	if n.Stmts != nil {
 		vv := v.GetChildrenVisitor("Stmts")
 		for _, nn := range n.Stmts {
-			nn.Walk(vv)
+			if nn != nil {
+				nn.Walk(vv)
+			}
 		}
 	}
 

node/expr/function_call.go

@@ -33,7 +33,9 @@ func (n *FunctionCall) Walk(v node.Visitor) {
 	if n.Arguments != nil {
 		vv := v.GetChildrenVisitor("Arguments")
 		for _, nn := range n.Arguments {
-			nn.Walk(vv)
+			if nn != nil {
+				nn.Walk(vv)
+			}
 		}
 	}
 

node/expr/isset.go

@@ -26,7 +26,9 @@ func (n *Isset) Walk(v node.Visitor) {
 	if n.Variables != nil {
 		vv := v.GetChildrenVisitor("Variables")
 		for _, nn := range n.Variables {
-			nn.Walk(vv)
+			if nn != nil {
+				nn.Walk(vv)
+			}
 		}
 	}
 

node/expr/list.go

@@ -26,7 +26,9 @@ func (n *List) Walk(v node.Visitor) {
 	if n.Items != nil {
 		vv := v.GetChildrenVisitor("Items")
 		for _, nn := range n.Items {
-			nn.Walk(vv)
+			if nn != nil {
+				nn.Walk(vv)
+			}
 		}
 	}
 

node/expr/method_call.go

@@ -40,7 +40,9 @@ func (n *MethodCall) Walk(v node.Visitor) {
 	if n.Arguments != nil {
 		vv := v.GetChildrenVisitor("Arguments")
 		for _, nn := range n.Arguments {
-			nn.Walk(vv)
+			if nn != nil {
+				nn.Walk(vv)
+			}
 		}
 	}
 

node/expr/new.go

@@ -33,7 +33,9 @@ func (n *New) Walk(v node.Visitor) {
 	if n.Arguments != nil {
 		vv := v.GetChildrenVisitor("Arguments")
 		for _, nn := range n.Arguments {
-			nn.Walk(vv)
+			if nn != nil {
+				nn.Walk(vv)
+			}
 		}
 	}
 

node/expr/shell_exec.go

@@ -26,7 +26,9 @@ func (n *ShellExec) Walk(v node.Visitor) {
 	if n.Parts != nil {
 		vv := v.GetChildrenVisitor("Parts")
 		for _, nn := range n.Parts {
-			nn.Walk(vv)
+			if nn != nil {
+				nn.Walk(vv)
+			}
 		}
 	}
 

node/expr/short_array.go

@@ -26,7 +26,9 @@ func (n *ShortArray) Walk(v node.Visitor) {
 	if n.Items != nil {
 		vv := v.GetChildrenVisitor("Items")
 		for _, nn := range n.Items {
-			nn.Walk(vv)
+			if nn != nil {
+				nn.Walk(vv)
+			}
 		}
 	}
 

node/expr/short_list.go

@@ -26,7 +26,9 @@ func (n *ShortList) Walk(v node.Visitor) {
 	if n.Items != nil {
 		vv := v.GetChildrenVisitor("Items")
 		for _, nn := range n.Items {
-			nn.Walk(vv)
+			if nn != nil {
+				nn.Walk(vv)
+			}
 		}
 	}
 

node/expr/static_call.go

@@ -40,7 +40,9 @@ func (n *StaticCall) Walk(v node.Visitor) {
 	if n.Arguments != nil {
 		vv := v.GetChildrenVisitor("Arguments")
 		for _, nn := range n.Arguments {
-			nn.Walk(vv)
+			if nn != nil {
+				nn.Walk(vv)
+			}
 		}
 	}