diff --git a/tracing/src/macros.rs b/tracing/src/macros.rs index 9f6bd7670..5de45cb75 100644 --- a/tracing/src/macros.rs +++ b/tracing/src/macros.rs @@ -1545,7 +1545,6 @@ macro_rules! trace { $crate::event!( target: module_path!(), $crate::Level::TRACE, - {}, $($arg)+ ) ); @@ -1822,7 +1821,6 @@ macro_rules! debug { $crate::event!( target: module_path!(), $crate::Level::DEBUG, - {}, $($arg)+ ) ); @@ -2110,7 +2108,6 @@ macro_rules! info { $crate::event!( target: module_path!(), $crate::Level::INFO, - {}, $($arg)+ ) ); @@ -2391,7 +2388,6 @@ macro_rules! warn { $crate::event!( target: module_path!(), $crate::Level::WARN, - {}, $($arg)+ ) ); @@ -2668,7 +2664,6 @@ macro_rules! error { $crate::event!( target: module_path!(), $crate::Level::ERROR, - {}, $($arg)+ ) ); diff --git a/tracing/tests/event.rs b/tracing/tests/event.rs index a370b700c..0af0602e9 100644 --- a/tracing/tests/event.rs +++ b/tracing/tests/event.rs @@ -416,6 +416,12 @@ fn constant_field_name() { ) }; let (collector, handle) = collector::mock() + .event(expect_event()) + .event(expect_event()) + .event(expect_event()) + .event(expect_event()) + .event(expect_event()) + .event(expect_event()) .event(expect_event()) .event(expect_event()) .only() @@ -439,6 +445,54 @@ fn constant_field_name() { }, "quux" ); + tracing::info!( + { std::convert::identity(FOO) } = "bar", + { "constant string" } = "also works", + foo.bar = "baz", + "quux" + ); + tracing::info!( + { + { std::convert::identity(FOO) } = "bar", + { "constant string" } = "also works", + foo.bar = "baz", + }, + "quux" + ); + tracing::event!( + Level::INFO, + { std::convert::identity(FOO) } = "bar", + { "constant string" } = "also works", + foo.bar = "baz", + "{}", + "quux" + ); + tracing::event!( + Level::INFO, + { + { std::convert::identity(FOO) } = "bar", + { "constant string" } = "also works", + foo.bar = "baz", + }, + "{}", + "quux" + ); + tracing::info!( + { std::convert::identity(FOO) } = "bar", + { "constant string" } = "also works", + foo.bar = "baz", + "{}", + "quux" + ); + tracing::info!( + { + { std::convert::identity(FOO) } = "bar", + { "constant string" } = "also works", + foo.bar = "baz", + }, + "{}", + "quux" + ); }); handle.assert_finished(); diff --git a/tracing/tests/macros.rs b/tracing/tests/macros.rs index 8d58bf610..a072389e2 100644 --- a/tracing/tests/macros.rs +++ b/tracing/tests/macros.rs @@ -555,12 +555,15 @@ fn trace() { trace!(foo = ?3, bar.baz = %2, quux = false); trace!(foo = 3, bar.baz = 2, quux = false); trace!(foo = 3, bar.baz = 3,); + trace!("foo" = 3, bar.baz = 3,); + trace!(foo = 3, "bar.baz" = 3,); trace!("foo"); trace!("foo: {}", 3); trace!(foo = ?3, bar.baz = %2, quux = false, "hello world {:?}", 42); trace!(foo = 3, bar.baz = 2, quux = false, "hello world {:?}", 42); trace!(foo = 3, bar.baz = 3, "hello world {:?}", 42,); trace!({ foo = 3, bar.baz = 80 }, "quux"); + trace!({ "foo" = 3, "bar.baz" = 80 }, "quux"); trace!({ foo = 2, bar.baz = 79 }, "quux {:?}", true); trace!({ foo = 2, bar.baz = 79 }, "quux {:?}, {quux}", true, quux = false); trace!({ foo = 2, bar.baz = 78 }, "quux"); @@ -579,6 +582,9 @@ fn trace() { trace!(?foo); trace!(%foo); trace!(foo); + trace!("foo" = ?foo); + trace!("foo" = %foo); + trace!("foo" = foo); trace!(name: "foo", ?foo); trace!(name: "foo", %foo); trace!(name: "foo", foo); @@ -615,12 +621,15 @@ fn debug() { debug!(foo = ?3, bar.baz = %2, quux = false); debug!(foo = 3, bar.baz = 2, quux = false); debug!(foo = 3, bar.baz = 3,); + debug!("foo" = 3, bar.baz = 3,); + debug!(foo = 3, "bar.baz" = 3,); debug!("foo"); debug!("foo: {}", 3); debug!(foo = ?3, bar.baz = %2, quux = false, "hello world {:?}", 42); debug!(foo = 3, bar.baz = 2, quux = false, "hello world {:?}", 42); debug!(foo = 3, bar.baz = 3, "hello world {:?}", 42,); debug!({ foo = 3, bar.baz = 80 }, "quux"); + debug!({ "foo" = 3, "bar.baz" = 80 }, "quux"); debug!({ foo = 2, bar.baz = 79 }, "quux {:?}", true); debug!({ foo = 2, bar.baz = 79 }, "quux {:?}, {quux}", true, quux = false); debug!({ foo = 2, bar.baz = 78 }, "quux"); @@ -639,6 +648,9 @@ fn debug() { debug!(?foo); debug!(%foo); debug!(foo); + debug!("foo" = ?foo); + debug!("foo" = %foo); + debug!("foo" = foo); debug!(name: "foo", ?foo); debug!(name: "foo", %foo); debug!(name: "foo", foo); @@ -675,12 +687,15 @@ fn info() { info!(foo = ?3, bar.baz = %2, quux = false); info!(foo = 3, bar.baz = 2, quux = false); info!(foo = 3, bar.baz = 3,); + info!("foo" = 3, bar.baz = 3,); + info!(foo = 3, "bar.baz" = 3,); info!("foo"); info!("foo: {}", 3); info!(foo = ?3, bar.baz = %2, quux = false, "hello world {:?}", 42); info!(foo = 3, bar.baz = 2, quux = false, "hello world {:?}", 42); info!(foo = 3, bar.baz = 3, "hello world {:?}", 42,); info!({ foo = 3, bar.baz = 80 }, "quux"); + info!({ "foo" = 3, "bar.baz" = 80 }, "quux"); info!({ foo = 2, bar.baz = 79 }, "quux {:?}", true); info!({ foo = 2, bar.baz = 79 }, "quux {:?}, {quux}", true, quux = false); info!({ foo = 2, bar.baz = 78 }, "quux"); @@ -699,6 +714,9 @@ fn info() { info!(?foo); info!(%foo); info!(foo); + info!("foo" = ?foo); + info!("foo" = %foo); + info!("foo" = foo); info!(name: "foo", ?foo); info!(name: "foo", %foo); info!(name: "foo", foo); @@ -735,12 +753,15 @@ fn warn() { warn!(foo = ?3, bar.baz = %2, quux = false); warn!(foo = 3, bar.baz = 2, quux = false); warn!(foo = 3, bar.baz = 3,); + warn!("foo" = 3, bar.baz = 3,); + warn!(foo = 3, "bar.baz" = 3,); warn!("foo"); warn!("foo: {}", 3); warn!(foo = ?3, bar.baz = %2, quux = false, "hello world {:?}", 42); warn!(foo = 3, bar.baz = 2, quux = false, "hello world {:?}", 42); warn!(foo = 3, bar.baz = 3, "hello world {:?}", 42,); warn!({ foo = 3, bar.baz = 80 }, "quux"); + warn!({ "foo" = 3, "bar.baz" = 80 }, "quux"); warn!({ foo = 2, bar.baz = 79 }, "quux {:?}", true); warn!({ foo = 2, bar.baz = 79 }, "quux {:?}, {quux}", true, quux = false); warn!({ foo = 2, bar.baz = 78 }, "quux"); @@ -759,6 +780,9 @@ fn warn() { warn!(?foo); warn!(%foo); warn!(foo); + warn!("foo" = ?foo); + warn!("foo" = %foo); + warn!("foo" = foo); warn!(name: "foo", ?foo); warn!(name: "foo", %foo); warn!(name: "foo", foo); @@ -795,15 +819,18 @@ fn error() { error!(foo = ?3, bar.baz = %2, quux = false); error!(foo = 3, bar.baz = 2, quux = false); error!(foo = 3, bar.baz = 3,); + error!("foo" = 3, bar.baz = 3,); + error!(foo = 3, "bar.baz" = 3,); error!("foo"); error!("foo: {}", 3); error!(foo = ?3, bar.baz = %2, quux = false, "hello world {:?}", 42); error!(foo = 3, bar.baz = 2, quux = false, "hello world {:?}", 42); error!(foo = 3, bar.baz = 3, "hello world {:?}", 42,); error!({ foo = 3, bar.baz = 80 }, "quux"); + error!({ "foo" = 3, "bar.baz" = 80 }, "quux"); error!({ foo = 2, bar.baz = 79 }, "quux {:?}", true); error!({ foo = 2, bar.baz = 79 }, "quux {:?}, {quux}", true, quux = false); - error!({ foo = 2, bar.baz = 78, }, "quux"); + error!({ foo = 2, bar.baz = 78 }, "quux"); error!({ foo = ?2, bar.baz = %78 }, "quux"); error!(name: "foo", foo = 3, bar.baz = 2, quux = false); error!(name: "foo", target: "foo_events", foo = 3, bar.baz = 2, quux = false); @@ -819,6 +846,9 @@ fn error() { error!(?foo); error!(%foo); error!(foo); + error!("foo" = ?foo); + error!("foo" = %foo); + error!("foo" = foo); error!(name: "foo", ?foo); error!(name: "foo", %foo); error!(name: "foo", foo);