@WG-prioritization/alerts issue #78532 has been requested for prioritization.
yeah, seems about right :thumbs_up: I fear that
VecDeque probably has more issues with ZSTs than just that though
there really shouldn't be a reason to use a queue of objects carrying no information, so we have close to no usage or test coverage here
I'll wait for a bit to see if anyone disagrees with the priority
Seems good to me, though maybe I'd have expected
P-medium sounds good to me, you'd have to do this intentionally and it's a panic, not a crash or unsoundness
Plus IIUC it's been there for a long time?
I still think P-high is better just to be on the safe side. It's possible there's some valid use case for a queue of ZSTs -- maybe something like Diesel's type magic
But I'm okay with P-medium as well
i am not completely certain that we can't cause ub here tbh
Ok, going with P-high then
Issue #78532's prioritization request has been removed.