make the second argument optional in FKNamed(name, inlined) (closes #8719)

src/macro/macroApi.ml

@@ -726,7 +726,9 @@ and decode_display_kind v = match (decode_enum v) with
 
 and decode_function_kind kind = if kind = vnull then FKAnonymous else match decode_enum kind with
 	| 0, [] -> FKAnonymous
-	| 1, [name;inline] -> FKNamed ((decode_string name,Globals.null_pos), decode_bool inline)
+	| 1, [name;inline] ->
+		let inline = if inline = vnull then false else decode_bool inline in
+		FKNamed ((decode_string name,Globals.null_pos), inline)
 	| 2, [] -> FKArrow
 	| _ -> raise Invalid_expr
 

std/haxe/macro/Expr.hx

@@ -393,7 +393,7 @@ enum FunctionKind {
 	/**
 		Named function
 	**/
-	FNamed(name:String, inlined:Bool);
+	FNamed(name:String, ?inlined:Bool);
 	/**
 		Arrow function
 	**/

tests/misc/projects/issue5002/Main2.hx

@@ -31,7 +31,7 @@ class Main2 {
 
 	static macro function invalidFunctionName() {
 		return {
-			expr: EFunction(FNamed("0_function", false), {
+			expr: EFunction(FNamed("0_function"), {
 				args: [],
 				ret: macro:Void,
 				expr: macro {}
@@ -42,7 +42,7 @@ class Main2 {
 
 	static macro function invalidFunctionArgumentName() {
 		return {
-			expr: EFunction(FNamed("foo", false), {
+			expr: EFunction(FNamed("foo"), {
 				args: [
 					{
 						name: "0_argument",