Extract hook export code

Author: ndossche

Zend/zend_ast.c

@@ -1744,6 +1744,51 @@ static ZEND_COLD void zend_ast_export_type(smart_str *str, zend_ast *ast, int in
 	zend_ast_export_ns_name(str, ast, 0, indent);
 }
 
+static ZEND_COLD void zend_ast_export_hook_list(smart_str *str, zend_ast_list *hook_list, int indent)
+{
+	smart_str_appends(str, " {");
+	smart_str_appendc(str, '\n');
+	indent++;
+	zend_ast_export_indent(str, indent);
+
+	for (uint32_t i = 0; i < hook_list->children; i++) {
+		zend_ast_decl *hook = (zend_ast_decl *)hook_list->child[i];
+		zend_ast_export_visibility(str, hook->flags, ZEND_MODIFIER_TARGET_PROPERTY);
+		if (hook->flags & ZEND_ACC_FINAL) {
+			smart_str_appends(str, "final ");
+		}
+		switch (i) {
+			case ZEND_PROPERTY_HOOK_GET:
+				smart_str_appends(str, "get");
+				break;
+			case ZEND_PROPERTY_HOOK_SET:
+				smart_str_appends(str, "set");
+				break;
+		}
+		zend_ast *body = hook->child[2];
+		if (body == NULL) {
+			smart_str_appendc(str, ';');
+		} else if (body->kind == ZEND_AST_PROPERTY_HOOK_SHORT_BODY) {
+			smart_str_appends(str, " => ");
+			zend_ast_export_ex(str, body->child[0], 0, indent);
+			smart_str_appendc(str, ';');
+		} else {
+			smart_str_appends(str, " {\n");
+			zend_ast_export_stmt(str, body, indent + 1);
+			zend_ast_export_indent(str, indent);
+			smart_str_appendc(str, '}');
+		}
+		if (i < (hook_list->children - 1)) {
+			smart_str_appendc(str, '\n');
+			zend_ast_export_indent(str, indent);
+		}
+	}
+	smart_str_appendc(str, '\n');
+	indent--;
+	zend_ast_export_indent(str, indent);
+	smart_str_appendc(str, '}');
+}
+
 #define BINARY_OP(_op, _p, _pl, _pr) do { \
 		op = _op; \
 		p = _p; \
@@ -2389,49 +2434,7 @@ static ZEND_COLD void zend_ast_export_ex(smart_str *str, zend_ast *ast, int prio
 			}
 
 			if (ast->child[3]) {
-				zend_ast_list *hook_list = zend_ast_get_list(ast->child[3]);
-
-				smart_str_appends(str, " {");
-				smart_str_appendc(str, '\n');
-				indent++;
-				zend_ast_export_indent(str, indent);
-
-				for (uint32_t i = 0; i < hook_list->children; i++) {
-					zend_ast_decl *hook = (zend_ast_decl *)hook_list->child[i];
-					zend_ast_export_visibility(str, hook->flags, ZEND_MODIFIER_TARGET_PROPERTY);
-					if (hook->flags & ZEND_ACC_FINAL) {
-						smart_str_appends(str, "final ");
-					}
-					switch (i) {
-						case ZEND_PROPERTY_HOOK_GET:
-							smart_str_appends(str, "get");
-							break;
-						case ZEND_PROPERTY_HOOK_SET:
-							smart_str_appends(str, "set");
-							break;
-					}
-					zend_ast *body = hook->child[2];
-					if (body == NULL) {
-						smart_str_appendc(str, ';');
-					} else if (body->kind == ZEND_AST_PROPERTY_HOOK_SHORT_BODY) {
-						smart_str_appends(str, " => ");
-						zend_ast_export_ex(str, body->child[0], 0, indent);
-						smart_str_appendc(str, ';');
-					} else {
-						smart_str_appends(str, " {\n");
-						zend_ast_export_stmt(str, body, indent + 1);
-						zend_ast_export_indent(str, indent);
-						smart_str_appendc(str, '}');
-					}
-					if (i < (hook_list->children - 1)) {
-						smart_str_appendc(str, '\n');
-						zend_ast_export_indent(str, indent);
-					}
-				}
-				smart_str_appendc(str, '\n');
-				indent--;
-				zend_ast_export_indent(str, indent);
-				smart_str_appendc(str, '}');
+				zend_ast_export_hook_list(str, zend_ast_get_list(ast->child[3]), indent);
 			}
 			break;
 		case ZEND_AST_CONST_ELEM: