Add is_constructing method to JS::CallArgs (#497)

saito828koki · kokisaito · web-flow · commit 4f0724dd3b9b · 2024-08-17T14:53:13.000Z
* feat: Add is_constructing to CallArgs which corresponds to `callargs_is_constructing` in servo repo

Signed-off-by: Koki Saito &lt;saitoto828@gmail.com&gt;

* fix: Use unsafe block

Signed-off-by: Koki Saito &lt;saitoto828@gmail.com&gt;

---------

Signed-off-by: Koki Saito &lt;saitoto828@gmail.com&gt;
Co-authored-by: kokisaito &lt;kosaito@indeed.com&gt;
diff --git a/mozjs-sys/src/jsimpls.rs b/mozjs-sys/src/jsimpls.rs
@@ -302,6 +302,11 @@ impl JS::CallArgs {
             JS::MutableHandleValue::from_marked_location(self.argv_.offset(self.argc_ as isize))
         }
     }
+
+    #[inline]
+    pub fn is_constructing(&self) -> bool {
+        unsafe { (*self.argv_.offset(-1)).is_magic() }
+    }
 }
 
 impl JSJitSetterCallArgs {

Original file line number	Diff line number	Diff line change
`@@ -302,6 +302,11 @@ impl JS::CallArgs {`
`302`	`302`	`JS::MutableHandleValue::from_marked_location(self.argv_.offset(self.argc_ as isize))`
`303`	`303`	`}`
`304`	`304`	`}`
	`305`	`+`
	`306`	`+ #[inline]`
	`307`	`+ pub fn is_constructing(&self) -> bool {`
	`308`	`+ unsafe { (*self.argv_.offset(-1)).is_magic() }`
	`309`	`+ }`
`305`	`310`	`}`
`306`	`311`
`307`	`312`	`impl JSJitSetterCallArgs {`