Merge pull request #635 from async-rs/revert-629-update-log

Revert "upgrade log, remove kv-log-macro"
async-rs · Dec 17, 2019 · d2c25f4 · d2c25f4
2 parents c7cf193 + d8befe2
commit d2c25f4
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 5 deletions.
diff --git a/Cargo.toml b/Cargo.toml
@@ -27,6 +27,7 @@ default = [
   "crossbeam-channel",
   "crossbeam-deque",
   "futures-timer",
+  "kv-log-macro",
   "log",
   "mio",
   "mio-uds",
@@ -57,7 +58,8 @@ crossbeam-utils = { version = "0.7.0", optional = true }
 futures-core = { version = "0.3.1", optional = true }
 futures-io = { version = "0.3.1", optional = true }
 futures-timer = { version = "2.0.2", optional = true }
-log = { version = "0.4.10", features = ["kv_unstable"], optional = true }
+kv-log-macro = { version = "1.0.4", optional = true }
+log = { version = "0.4.8", features = ["kv_unstable"], optional = true }
 memchr = { version = "2.2.1", optional = true }
 mio = { version = "0.6.19", optional = true }
 mio-uds = { version = "0.6.7", optional = true }

diff --git a/src/task/block_on.rs b/src/task/block_on.rs
@@ -6,6 +6,7 @@ use std::task::{RawWaker, RawWakerVTable};
 use std::thread;
 
 use crossbeam_utils::sync::Parker;
+use kv_log_macro::trace;
 use log::log_enabled;
 
 use crate::task::{Context, Poll, Task, Waker};
@@ -42,7 +43,7 @@ where
 
     // Log this `block_on` operation.
     if log_enabled!(log::Level::Trace) {
-        log::trace!("block_on", {
+        trace!("block_on", {
             task_id: task.id().0,
             parent_task_id: Task::get_current(|t| t.id().0).unwrap_or(0),
         });
@@ -58,7 +59,7 @@ where
         defer! {
             if log_enabled!(log::Level::Trace) {
                 Task::get_current(|t| {
-                    log::trace!("completed", {
+                    trace!("completed", {
                         task_id: t.id().0,
                     });
                 });

diff --git a/src/task/builder.rs b/src/task/builder.rs
@@ -1,3 +1,4 @@
+use kv_log_macro::trace;
 use log::log_enabled;
 use std::future::Future;
 
@@ -37,7 +38,7 @@ impl Builder {
 
         // Log this `spawn` operation.
         if log_enabled!(log::Level::Trace) {
-            log::trace!("spawn", {
+            trace!("spawn", {
                 task_id: task.id().0,
                 parent_task_id: Task::get_current(|t| t.id().0).unwrap_or(0),
             });
@@ -53,7 +54,7 @@ impl Builder {
             defer! {
                 if log_enabled!(log::Level::Trace) {
                     Task::get_current(|t| {
-                        log::trace!("completed", {
+                        trace!("completed", {
                             task_id: t.id().0,
                         });
                     });