Re: [PATCH] rust: time: implement `Display` for `Delta`
From: Miguel Ojeda
Date: Wed May 13 2026 - 13:51:39 EST
On Mon, May 11, 2026 at 4:23 PM Andreas Hindborg <a.hindborg@xxxxxxxxxx> wrote:
>
> Please advise which of the following make most sense:
>
> - Merge this patch without a suffix.
> - Iterate on this patch and merge it without a user.
> - Drop this patch for now.
Having `ns` may be a good idea.
In fact, if you do that, you could consider using separators every few
digits if the number is too large (like Rust integer literals allow
with underscores), e.g. `12_345ns`
Or you could even consider picking other units depending on the
magnitude, e.g. `12.345ms`. After all, if we are going for human
readability, then there is an argument to be made for that. Even about
showing only a few significant figures, e.g. `12.3ms`. Anyone that
needs more should probably be doing something manually with the nanos
anyway.
But it is up to you and timekeeping, of course. I guess it should be
decided depending on the expected users and what most of them would
want to see in the majority of cases.
Cheers,
Miguel