fix: fix JsPropertyDescriptor

HosihikariAndMore · Aug 2, 2023 · 80b9329 · 80b9329
1 parent 01a7718
commit 80b9329
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/src/Modules/Clr/Main.cs b/src/Modules/Clr/Main.cs
@@ -120,8 +120,8 @@ JsAtom propName
         { //todo fix this
             data = new JsPropertyDescriptor
             {
-                Flags = JsPropertyFlags.HasGet,
-                Getter = ctxInstance
+                Flags = JsPropertyFlags.HasValue,
+                Value = ctxInstance
                     .NewJsFunctionObject(
                         (_, thisObj, argv) =>
                         {

diff --git a/src/QuickJS/Types/JsPropertyDescriptor.cs b/src/QuickJS/Types/JsPropertyDescriptor.cs
@@ -2,7 +2,7 @@
 
 namespace Hosihikari.VanillaScript.QuickJS.Types;
 
-[StructLayout(LayoutKind.Explicit, Size = 16 * 4)]
+[StructLayout(LayoutKind.Explicit, Size = 16 * 3 + 8)]
 public ref struct JsPropertyDescriptor
 {
     /*
@@ -17,12 +17,12 @@ typedef struct JSPropertyDescriptor {
     public JsPropertyFlags Flags;
 
     //ref to js_free_desc(__int64 a1, _QWORD *a2)
-    [FieldOffset(16)]
+    [FieldOffset(8)]
     public JsValue Value;
 
-    [FieldOffset(2 * 16)]
+    [FieldOffset(8 + 16)]
     public JsValue Getter;
 
-    [FieldOffset(3 * 16)]
+    [FieldOffset(8 + 16 + 16)]
     public JsValue Setter;
 }