Merge pull request #416 from chewing/rust/tracing

refactor: enable verbose tracing for Debug profile

CMakeLists.txt

@@ -80,7 +80,11 @@ option(USE_VALGRIND "Use valgrind when testing" true)
 option(WITH_RUST "Use rust implemented internals (experimental)" false)
 if (WITH_RUST)
     add_subdirectory(cmake/corrosion)
-    corrosion_import_crate(MANIFEST_PATH Cargo.toml CRATES chewing FEATURES capi)
+    if (${CMAKE_BUILD_TYPE} STREQUAL "Debug")
+        corrosion_import_crate(MANIFEST_PATH Cargo.toml CRATES chewing FEATURES capi test-tracing)
+    else()
+        corrosion_import_crate(MANIFEST_PATH Cargo.toml CRATES chewing FEATURES capi)
+    endif()
     corrosion_import_crate(MANIFEST_PATH Cargo.toml CRATES chewing-tools)
     add_compile_definitions(WITH_RUST)
 endif()

Cargo.toml

@@ -10,13 +10,19 @@ edition = "2021"
 include = ["src/**/*.rs", "Cargo.toml", "AUTHORS", "COPYING", "NEWS"]
 
 [dependencies]
-bytemuck = { version = "1.13.1", features = ["derive"] }
+bytemuck = { version = "1.14.0", features = ["derive"] }
 directories = "5.0.0"
-indexmap = "2.0.0"
+indexmap = "2.1.0"
 riff = "2.0.0"
 rusqlite = "0.30.0"
 thiserror = "1.0.0"
-tracing = "0.1.37"
+tracing = { version = "0.1.40", features = [
+    "max_level_trace",
+    "release_max_level_warn",
+] }
+tracing-subscriber = { version = "0.3.18", features = [
+    "env-filter",
+], optional = true }
 
 [target.'cfg(windows)'.dependencies]
 rusqlite = { version = "0.30.0", features = ["bundled"] }
@@ -27,6 +33,7 @@ crate-type = ["lib", "staticlib"]
 [features]
 default = []
 capi = ["cbindgen"]
+test-tracing = ["tracing-subscriber"]
 
 [build-dependencies]
 # https://github.com/mozilla/cbindgen/issues/891

src/capi/io.rs

@@ -728,7 +728,8 @@ pub extern "C" fn chewing_set_logger(
     logger: extern "C" fn(data: *mut c_void, level: c_int, fmt: *const c_char, arg: ...),
     data: *mut c_void,
 ) {
-    // let _ = tracing_subscriber::fmt::try_init();
+    #[cfg(all(feature = "test-tracing", debug_assertions))]
+    let _ = tracing_subscriber::fmt::try_init();
 }
 
 #[tracing::instrument(skip(ctx), ret)]