Conversation
Similar to the other `set_` methods -- it's possible to do this as `bar.clone().with_elapsed` but not very intuitive what that means.
There's a few spots in nextest that need to rewind the elapsed time to an earlier point. One example is https://github.com/nextest-rs/nextest/blob/83952e13433fa86c24e2cdef1aaddeacb6dd85f4/nextest-runner/src/reporter/displayer/progress.rs#L114 where after SIGTSTP and SIGCONT, elapsed timers need to be moved back by however long the invocation was paused. Right now I do this through clone + with_elapsed, but it's pretty non-intuitive that it does the right thing. |
|
The lint issue looks like something new in Rust 1.91, unrelated to the PR. |
djc
left a comment
There was a problem hiding this comment.
Thanks!
Would you like a release with this?
If you have the time, sure, but it's not a huge rush since the clone workaround works fine. |
|
Will release after is merged. |
Similar to the other
set_methods -- it's possible to do this asbar.clone().with_elapsedbut not very intuitive what that means.