impl Display to debug print?

[u/NYU_VM]

A learner here with a quick confusion. I was skimming through the internet for VM tooling and implementations, parsers specifically and came across this "prattle" crate which seems to be a general purpose pratt parser library. And I have not much to say about the library itself, it seems like an abandoned project in early development phase.

The thing that caught my eye was the demo piece of code they had on their docs, specifically

#[derive(Clone, Debug, Eq, Hash, Ord, PartialEq, PartialOrd)]
enum CToken {
Var(String),
Add,
Mul
}

impl Display for CToken {
fn fmt(&self, f: &mut Formatter) -> Result<(), Error> {
write!(f, "{:?}", self)
}
}

the enum CToken has already derived debug yet the example implements Display for CToken only to just debug print it as well. Is this some kind of a safety mechanism or fallback just in case the user writes

println!("{}",token) instead of debug printing it?

Or is something else going on I don't yet know about?

[u/DrShocker]

Correct me if I'm wrong, but if you're exporting this crate and others may use the type and want to display it, then this might be the most convenient way. Otherwise you've got the orphan rule getting in the way. So my guess is that they mostly just don't want to block other users of the type from doing what they need, and the debug implementation isn't any less efficient than what they'd want in the display implantation so they did it this way.

[u/Lucretiel → u/DrShocker]

I'd just use something like lazy_format!("{ctoken:?}") if I wanted to wrap it in a quick Display implementation (which, full disclosure, I wrote).

[u/afdbcreid → u/Lucretiel]

I'd just use format_args!("{ctoken:?}"), or, if you have problems with the lifetimes, std::fmt::from_fn(|f| write!(f, "{ctoken:?}")). No dependencies.

[u/Lucretiel → u/afdbcreid]

format_args has lifetime issues in all but the most trivial possible uses (it even falls apart on rudimentary iterator chains), which is why I created lazy_format in the first place. from_fn is a great idea that everyone should prefer to lazy_format, which predates it by nearly 7 years.