Bläddra i källkod

Don't require Realm in Constructor public constructor (#1441)

Marko Lahma 2 år sedan
förälder
incheckning
fedd68033b
2 ändrade filer med 6 tillägg och 2 borttagningar
  1. 1 1
      Jint.Tests.PublicInterface/ConstructorTests.cs
  2. 5 1
      Jint/Native/Constructor.cs

+ 1 - 1
Jint.Tests.PublicInterface/ConstructorTests.cs

@@ -54,7 +54,7 @@ public class ConstructorTests
 
 
 file sealed class DateOnlyConstructor : Constructor
 file sealed class DateOnlyConstructor : Constructor
 {
 {
-    public DateOnlyConstructor(Engine engine) : base(engine, engine.Realm, (JsString) "DateOnly")
+    public DateOnlyConstructor(Engine engine) : base(engine, "DateOnly")
     {
     {
     }
     }
 
 

+ 5 - 1
Jint/Native/Constructor.cs

@@ -6,7 +6,11 @@ namespace Jint.Native;
 
 
 public abstract class Constructor : FunctionInstance, IConstructor
 public abstract class Constructor : FunctionInstance, IConstructor
 {
 {
-    protected Constructor(Engine engine, Realm realm, JsString name) : base(engine, realm, name)
+    protected Constructor(Engine engine, string name) : this(engine, engine.Realm, new JsString(name))
+    {
+    }
+
+    internal Constructor(Engine engine, Realm realm, JsString name) : base(engine, realm, name)
     {
     {
     }
     }