feat: upgrade quickjs-ng to v0.9.0 and fix compatibility

Author: Icemic

examples/catch_promise_rejection.rs

@@ -51,7 +51,7 @@ unsafe extern "C" fn host_promise_rejection_tracker(
     ctx: *mut JSContext,
     _promise: RawJSValue,
     reason: RawJSValue,
-    is_handled: c_int,
+    is_handled: bool,
     _opaque: *mut c_void,
 ) {
     let reason = OwnedJsValue::own(ctx, &reason);

libquickjs-sys/embed/extensions.c

@@ -62,7 +62,7 @@ JSValue JS_Ext_NewBool(JSContext *ctx, uint8_t val)
     return JS_NewBool(ctx, val);
 }
 
-JS_BOOL JS_Ext_IsNan(JSValue v)
+bool JS_Ext_IsNan(JSValue v)
 {
     return JS_VALUE_IS_NAN(v);
 }
@@ -92,12 +92,12 @@ int JS_Ext_GetNormTag(JSValue v)
     return JS_VALUE_GET_NORM_TAG(v);
 }
 
-JS_BOOL JS_Ext_IsNumber(JSValue v)
+bool JS_Ext_IsNumber(JSValue v)
 {
     return JS_IsNumber(v);
 }
 
-JS_BOOL JS_Ext_IsBigInt(JSContext *ctx, JSValue v)
+bool JS_Ext_IsBigInt(JSContext *ctx, JSValue v)
 {
     return JS_IsBigInt(ctx, v);
 }
@@ -110,42 +110,42 @@ JS_BOOL JS_Ext_IsBigInt(JSContext *ctx, JSValue v)
 //     return JS_IsBigDecimal(v);
 // }
 
-JS_BOOL JS_Ext_IsBool(JSValue v)
+bool JS_Ext_IsBool(JSValue v)
 {
     return JS_IsBool(v);
 }
 
-JS_BOOL JS_Ext_IsNull(JSValue v)
+bool JS_Ext_IsNull(JSValue v)
 {
     return JS_IsNull(v);
 }
 
-JS_BOOL JS_Ext_IsUndefined(JSValue v)
+bool JS_Ext_IsUndefined(JSValue v)
 {
     return JS_IsUndefined(v);
 }
 
-JS_BOOL JS_Ext_IsException(JSValue v)
+bool JS_Ext_IsException(JSValue v)
 {
     return JS_IsException(v);
 }
 
-JS_BOOL JS_Ext_IsUninitialized(JSValue v)
+bool JS_Ext_IsUninitialized(JSValue v)
 {
     return JS_IsUninitialized(v);
 }
 
-JS_BOOL JS_Ext_IsString(JSValue v)
+bool JS_Ext_IsString(JSValue v)
 {
     return JS_IsString(v);
 }
 
-JS_BOOL JS_Ext_IsSymbol(JSValue v)
+bool JS_Ext_IsSymbol(JSValue v)
 {
     return JS_IsSymbol(v);
 }
 
-JS_BOOL JS_Ext_IsObject(JSValue v)
+bool JS_Ext_IsObject(JSValue v)
 {
     return JS_IsObject(v);
 }
@@ -173,7 +173,7 @@ JSValue JS_Ext_NewCFunctionMagic(JSContext *ctx,
     return JS_NewCFunctionMagic(ctx, func, name, length, cproto, magic);
 }
 
-JS_BOOL JS_Ext_IsPromise(JSContext *ctx, JSValue val)
+bool JS_Ext_IsPromise(JSContext *ctx, JSValue val)
 {
     void *p = JS_GetOpaque(val, JS_CLASS_PROMISE);
     return p != NULL;

libquickjs-sys/embed/extensions.h

@@ -22,23 +22,23 @@ extern "C"
   JSValue JS_Ext_NewInt32(JSContext *ctx, int32_t val);
   JSValue JS_Ext_NewBool(JSContext *ctx, uint8_t val);
 
-  JS_BOOL JS_Ext_IsNan(JSValue v);
+  bool JS_Ext_IsNan(JSValue v);
   double JS_Ext_GetFloat64(JSValue v);
   int JS_Ext_GetInt(JSValue v);
   int JS_Ext_GetBool(JSValue v);
   void *JS_Ext_GetPtr(JSValue v);
   int JS_Ext_GetNormTag(JSValue v);
 
-  JS_BOOL JS_Ext_IsNumber(JSValue v);
-  JS_BOOL JS_Ext_IsBigInt(JSContext *ctx, JSValue v);
-  JS_BOOL JS_Ext_IsBool(JSValue v);
-  JS_BOOL JS_Ext_IsNull(JSValue v);
-  JS_BOOL JS_Ext_IsUndefined(JSValue v);
-  JS_BOOL JS_Ext_IsException(JSValue v);
-  JS_BOOL JS_Ext_IsUninitialized(JSValue v);
-  JS_BOOL JS_Ext_IsString(JSValue v);
-  JS_BOOL JS_Ext_IsSymbol(JSValue v);
-  JS_BOOL JS_Ext_IsObject(JSValue v);
+  bool JS_Ext_IsNumber(JSValue v);
+  bool JS_Ext_IsBigInt(JSContext *ctx, JSValue v);
+  bool JS_Ext_IsBool(JSValue v);
+  bool JS_Ext_IsNull(JSValue v);
+  bool JS_Ext_IsUndefined(JSValue v);
+  bool JS_Ext_IsException(JSValue v);
+  bool JS_Ext_IsUninitialized(JSValue v);
+  bool JS_Ext_IsString(JSValue v);
+  bool JS_Ext_IsSymbol(JSValue v);
+  bool JS_Ext_IsObject(JSValue v);
 
   int JS_Ext_ToUint32(JSContext *ctx, uint32_t *pres, JSValue val);
   JSValue JS_Ext_NewCFunction(JSContext *ctx,
@@ -51,7 +51,7 @@ extern "C"
                                    int length,
                                    JSCFunctionEnum cproto,
                                    int magic);
-  JS_BOOL JS_Ext_IsPromise(JSContext *ctx, JSValue val);
+  bool JS_Ext_IsPromise(JSContext *ctx, JSValue val);
 
   JSValue JS_Ext_PromiseResolve(JSContext *ctx, JSValue value);
   JSValue JS_Ext_PromiseReject(JSContext *ctx, JSValue value);

libquickjs-sys/embed/quickjs

@@ -16,7 +16,7 @@ pub(crate) fn deserialize_borrowed_str(
 
     match tag {
         q::JS_TAG_STRING => {
-            let ptr = unsafe { q::JS_ToCStringLen2(context, std::ptr::null_mut(), *r, 0) };
+            let ptr = unsafe { q::JS_ToCStringLen2(context, std::ptr::null_mut(), *r, false) };
 
             if ptr.is_null() {
                 return Err(ValueError::Internal(
@@ -300,7 +300,7 @@ use crate::ExecutionError;
 
 /// Get the last exception from the runtime, and if present, convert it to a ExceptionError.
 pub(crate) fn get_exception(context: *mut q::JSContext) -> Option<ExecutionError> {
-    if unsafe { q::JS_HasException(context) } == 0 {
+    if unsafe { !q::JS_HasException(context) } {
         return None;
     }
 

src/utils.rs

@@ -147,19 +147,19 @@ impl OwnedJsValue {
     /// Check if this value is a Javascript array.
     #[inline]
     pub fn is_array(&self) -> bool {
-        unsafe { q::JS_IsArray(self.context, self.value) == 1 }
+        unsafe { q::JS_IsArray(self.value) }
     }
 
     /// Check if this value is a Javascript function.
     #[inline]
     pub fn is_function(&self) -> bool {
-        unsafe { q::JS_IsFunction(self.context, self.value) == 1 }
+        unsafe { q::JS_IsFunction(self.context, self.value) }
     }
 
     /// Check if this value is a Javascript promise.
     #[inline]
     pub fn is_promise(&self) -> bool {
-        unsafe { q::JS_Ext_IsPromise(self.context, self.value) == 1 }
+        unsafe { q::JS_Ext_IsPromise(self.context, self.value) }
     }
 
     /// Check if this value is a Javascript module.
@@ -213,7 +213,8 @@ impl OwnedJsValue {
     /// Convert this value into a string
     pub fn to_string(&self) -> Result<String, ValueError> {
         self.check_tag(JsTag::String)?;
-        let ptr = unsafe { q::JS_ToCStringLen2(self.context, std::ptr::null_mut(), self.value, 0) };
+        let ptr =
+            unsafe { q::JS_ToCStringLen2(self.context, std::ptr::null_mut(), self.value, false) };
 
         if ptr.is_null() {
             return Err(ValueError::Internal(