Maybe you should add a publicly-unconstructible (outside of this module) `pub struct DelaySpanBugEmitted(());` field to `Error`, even before we change anything about `ErrorReported` itself.    If nothing else, it would catch code that does this (or the const thing, which is probably even more plausible) instead of calling the method.