Clean up in code documentation

Author: gyuheon0h

datadog-crashtracker-ffi/src/runtime_callback.rs

@@ -15,9 +15,7 @@ use datadog_crashtracker::{
 pub use datadog_crashtracker::CallbackType as ddog_CallbackType;
 pub use datadog_crashtracker::RuntimeType as ddog_RuntimeType;
 
-#[cfg(test)]
-use datadog_crashtracker::clear_runtime_callback;
-use std::ffi::c_char;
+pub use datadog_crashtracker::RuntimeStackFrame as ddog_RuntimeStackFrame;
 
 /// Result type for runtime callback registration
 #[repr(C)]
@@ -60,7 +58,7 @@ impl From<CallbackError> for CallbackResult {
 /// static void my_runtime_callback(
 ///     void (*emit_frame)(const ddog_RuntimeStackFrame*),
 ///     void (*emit_stacktrace_string)(const char*),
-///     void* context
+///     void* writer_ctx
 /// ) {
 ///     // Collect runtime frames and call emit_frame for each one
 ///     ddog_RuntimeStackFrame frame = {
@@ -71,7 +69,7 @@ impl From<CallbackError> for CallbackResult {
 ///         .class_name = "MyClass",
 ///         .module_name = NULL
 ///     };
-///     emit_frame(&frame);
+///     emit_frame(writer_ctx, &frame);
 /// }
 ///
 ///
@@ -131,7 +129,7 @@ pub extern "C" fn ddog_crasht_is_runtime_callback_registered() -> bool {
 /// - The caller should not free the returned pointer
 /// - The returned string should be copied if it needs to persist beyond callback lifetime
 #[no_mangle]
-pub unsafe extern "C" fn ddog_crasht_get_registered_runtime_type() -> *const c_char {
+pub unsafe extern "C" fn ddog_crasht_get_registered_runtime_type() -> *const std::ffi::c_char {
     get_registered_runtime_type_ptr()
 }
 
@@ -145,19 +143,15 @@ pub unsafe extern "C" fn ddog_crasht_get_registered_runtime_type() -> *const c_c
 /// - The caller should not free the returned pointer
 /// - The returned string should be copied if it needs to persist beyond callback lifetime
 #[no_mangle]
-pub unsafe extern "C" fn ddog_crasht_get_registered_callback_type() -> *const c_char {
+pub unsafe extern "C" fn ddog_crasht_get_registered_callback_type() -> *const std::ffi::c_char {
     get_registered_callback_type_ptr()
 }
 
-// Re-export the core RuntimeStackFrame for C interop; consumers need to create RuntimeStackFrame
-pub use datadog_crashtracker::RuntimeStackFrame as ddog_RuntimeStackFrame;
-
 #[cfg(test)]
 mod tests {
     use super::*;
-    use datadog_crashtracker::RuntimeStackFrame;
-    use std::ffi::c_void;
-    use std::ffi::CString;
+    use datadog_crashtracker::{clear_runtime_callback, RuntimeStackFrame};
+    use std::ffi::{c_char, c_void, CString};
     use std::ptr;
     use std::sync::Mutex;