clippy fixes

HiRoFa · Jan 3, 2024 · 438c4ee · 438c4ee
1 parent fbb7277
commit 438c4ee
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/src/features/console.rs b/src/features/console.rs
@@ -176,7 +176,7 @@ unsafe fn parse_line(ctx: *mut q::JSContext, args: Vec<QuickJsValueAdapter>) ->
 
     output.push_str("JS_REALM:");
     QuickJsRealmAdapter::with_context(ctx, |realm| {
-        output.push_str("[");
+        output.push('[');
         output.push_str(realm.id.as_str());
         output.push_str("][");
         if let Ok(script_or_module_name) = quickjs_utils::get_script_or_module_name_q(realm) {

diff --git a/src/reflection/mod.rs b/src/reflection/mod.rs
@@ -724,6 +724,9 @@ pub fn get_proxy_instance_id_q(
     }
 }
 
+/// Get the instance id of a proxy instance
+/// # Safety
+/// please make sure context is still valid
 pub unsafe fn get_proxy_instance_id(
     ctx: *mut libquickjs_sys::JSContext,
     obj: &QuickJsValueAdapter,
@@ -736,6 +739,7 @@ pub unsafe fn get_proxy_instance_id(
     }
 }
 
+
 pub fn is_proxy_instance_q(q_ctx: &QuickJsRealmAdapter, obj: &QuickJsValueAdapter) -> bool {
     unsafe { is_proxy_instance(q_ctx.context, obj) }
 }