From 5105293ce6438daa7096b658ee26bfa561a0b3f8 Mon Sep 17 00:00:00 2001
From: mappu <mappu04@gmail.com>
Date: Sat, 11 Apr 2020 12:46:28 +1200
Subject: [PATCH] node: basic support for *expr.Closure

---
 fixtures/0013-closures.php |  9 +++++++++
 node.go                    | 18 ++++++++++++++++++
 2 files changed, 27 insertions(+)
 create mode 100644 fixtures/0013-closures.php

diff --git a/fixtures/0013-closures.php b/fixtures/0013-closures.php
new file mode 100644
index 0000000..df29624
--- /dev/null
+++ b/fixtures/0013-closures.php
@@ -0,0 +1,9 @@
+<?php
+
+const FOO = 'X';
+
+$x = function(string $param): string {
+	return $param . FOO;
+};
+
+echo $x("x");
diff --git a/node.go b/node.go
index a5f26d7..3948e52 100644
--- a/node.go
+++ b/node.go
@@ -1036,6 +1036,24 @@ func (this *conversionState) convertNoFreeFloating(n_ node.Node) (string, error)
 
 		return className + constName, nil
 
+	case *expr.Closure:
+		// Need to decide what to do with `this` binding
+		// TODO figure out what it means to use `this` in all types of nested closures
+
+		if n.Static {
+			return "", parseErr{n, fmt.Errorf("use of `static` in closure implies `this`-insensitive, but we can't make that true")}
+		}
+
+		// TODO n.PhpDocComment
+		// TODO n.ClosureUse
+
+		body, err := this.convertFunctionCommon(n.Params, n.ReturnType, n.ReturnsRef, n.Stmts)
+		if err != nil {
+			return "", parseErr{n, err}
+		}
+
+		return "(func" + strings.TrimRight(body, " \n") + ")", nil
+
 	case *expr.Exit:
 		// die(0) - set process exit code and exit
 		// die("message") - print to stdout and exit 0