Skip to content

Commit

Permalink
change ret(Debug) to ret where possible
Browse files Browse the repository at this point in the history
Now that `ret` displays `Result::Ok` but not `Result`, there are some
case that `ret(Debug)` is not necessary any longer.
  • Loading branch information
hkmatsumoto committed Nov 17, 2021
1 parent d049d80 commit a12814d
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions tracing-attributes/tests/ret.rs
Original file line number Diff line number Diff line change
Expand Up @@ -167,7 +167,7 @@ fn test_dbg() {
handle.assert_finished();
}

#[instrument(err, ret(Debug))]
#[instrument(err, ret)]
fn ret_and_err() -> Result<u8, TryFromIntError> {
u8::try_from(1234)
}
Expand Down Expand Up @@ -196,7 +196,7 @@ fn test_ret_and_err() {
handle.assert_finished();
}

#[instrument(err, ret(Debug))]
#[instrument(err, ret)]
fn ret_and_ok() -> Result<u8, TryFromIntError> {
u8::try_from(123)
}
Expand Down

0 comments on commit a12814d

Please sign in to comment.