Added Function factory

yantrajs · Aug 30, 2024 · 20cc5e5 · 20cc5e5
1 parent 9b3a7f6
commit 20cc5e5
Show file tree

Hide file tree

Showing 2 changed files with 40 additions and 24 deletions.
diff --git a/YantraJS.Core/Core/Function/JSFieldInfo.cs b/YantraJS.Core/Core/Function/JSFieldInfo.cs
@@ -23,41 +23,47 @@ public JSFieldInfo(ClrMemberNamingConvention namingConvention, FieldInfo field)
 
         public JSFunction GenerateFieldGetter()
         {
-            var args = Expression.Parameter(typeof(Arguments).MakeByRefType());
-            Expression convertedThis = field.IsStatic
-                ? null
-                : JSValueToClrConverter.Get(ArgumentsBuilder.This(args), field.DeclaringType, "this");
-            var body =
-                ClrProxyBuilder.Marshal(
-                    Expression.Field(
-                        convertedThis, field));
             var name = $"get {Name}";
-            var lambda = Expression.Lambda<JSFunctionDelegate>(name, body, args);
-            return new JSFunction(lambda.Compile(), name);
+            var field = this.field;
+            return new JSFunction(() => {
+                var args = Expression.Parameter(typeof(Arguments).MakeByRefType());
+                Expression convertedThis = field.IsStatic
+                    ? null
+                    : JSValueToClrConverter.Get(ArgumentsBuilder.This(args), field.DeclaringType, "this");
+                var body =
+                    ClrProxyBuilder.Marshal(
+                        Expression.Field(
+                            convertedThis, field));
+                var lambda = Expression.Lambda<JSFunctionDelegate>(name, body, args);
+                return lambda.Compile();
+            }, name);
 
         }
 
         public JSFunction GenerateFieldSetter()
         {
-            var args = Expression.Parameter(typeof(Arguments).MakeByRefType());
-            var a1 = ArgumentsBuilder.Get1(args);
-            var convert = field.IsStatic
-                ? null
-                : JSValueToClrConverter.Get(ArgumentsBuilder.This(args), field.DeclaringType, "this");
+            var name = $"set {Name}";
+            var field = this.field;
+            return new JSFunction(() => {
+                var args = Expression.Parameter(typeof(Arguments).MakeByRefType());
+                var a1 = ArgumentsBuilder.Get1(args);
+                var convert = field.IsStatic
+                    ? null
+                    : JSValueToClrConverter.Get(ArgumentsBuilder.This(args), field.DeclaringType, "this");
 
-            var clrArg1 = JSValueToClrConverter.Get(a1, field.FieldType, "value");
+                var clrArg1 = JSValueToClrConverter.Get(a1, field.FieldType, "value");
 
 
-            var fieldExp = Expression.Field(convert, field);
+                var fieldExp = Expression.Field(convert, field);
 
-            // todo
-            // not working for `char`
-            var assign = Expression.Assign(fieldExp, clrArg1).ToJSValue();
+                // todo
+                // not working for `char`
+                var assign = Expression.Assign(fieldExp, clrArg1).ToJSValue();
 
-            var body = assign;
-            var name = $"set {Name}";
-            var lambda = Expression.Lambda<JSFunctionDelegate>(name, body, args);
-            return new JSFunction(lambda.Compile(), name);
+                var body = assign;
+                var lambda = Expression.Lambda<JSFunctionDelegate>(name, body, args);
+                return lambda.Compile();
+            }, name);
         }
     }
 }
diff --git a/YantraJS.Core/Core/Function/JSFunction.cs b/YantraJS.Core/Core/Function/JSFunction.cs
@@ -121,6 +121,16 @@ public JSFunction(JSFunctionDelegate f)
 
         }
 
+        public JSFunction(Func<JSFunctionDelegate> fx, in StringSpan name)
+            : this(empty, in name, StringSpan.Empty)
+        {
+            this.f = (in Arguments a) =>
+            {
+                this.f = fx();
+                return this.f(in a);
+            };
+        }
+
         public JSFunction(
             JSFunctionDelegate f,
             in StringSpan name,