Jelajahi Sumber

Rename WeakRefInstance to JsWeakRef (#1602)

Marko Lahma 2 tahun lalu
induk
melakukan
0b1ed1b394

+ 2 - 2
Jint/Native/WeakRef/WeakRefInstance.cs → Jint/Native/WeakRef/JsWeakRef.cs

@@ -5,11 +5,11 @@ namespace Jint.Native.WeakRef;
 /// <summary>
 /// https://tc39.es/ecma262/#sec-properties-of-weak-ref-instances
 /// </summary>
-internal sealed class WeakRefInstance : ObjectInstance
+internal sealed class JsWeakRef : ObjectInstance
 {
     private readonly WeakReference<JsValue> _weakRefTarget;
 
-    public WeakRefInstance(Engine engine, JsValue target) : base(engine)
+    public JsWeakRef(Engine engine, JsValue target) : base(engine)
     {
         _weakRefTarget = new WeakReference<JsValue>(target);
     }

+ 1 - 1
Jint/Native/WeakRef/WeakRefConstructor.cs

@@ -44,7 +44,7 @@ internal sealed class WeakRefConstructor : Constructor
         var weakRef = OrdinaryCreateFromConstructor(
             newTarget,
             static intrinsics => intrinsics.WeakRef.PrototypeObject,
-            static (engine, _, target) => new WeakRefInstance(engine, target!),
+            static (engine, _, target) => new JsWeakRef(engine, target!),
             target);
 
         _engine.AddToKeptObjects(target);

+ 4 - 4
Jint/Native/WeakRef/WeakRefPrototype.cs

@@ -43,12 +43,12 @@ internal sealed class WeakRefPrototype : Prototype
 
     private JsValue Deref(JsValue thisObject, JsValue[] arguments)
     {
-        var weakRef = thisObject as WeakRefInstance;
-        if (weakRef is null)
+        if (thisObject is JsWeakRef weakRef)
         {
-            ExceptionHelper.ThrowTypeError(_realm, "object must be a WeakRef");
+            return weakRef.WeakRefDeref();
         }
 
-        return weakRef.WeakRefDeref();
+        ExceptionHelper.ThrowTypeError(_realm, "object must be a WeakRef");
+        return default;
     }
 }