Re: [PATCH] rust: arc: remove unused PhantomData
From: Miguel Ojeda
Date: Wed Nov 06 2024 - 13:30:32 EST
On Wed, Nov 6, 2024 at 5:41 PM Tamir Duberstein <tamird@xxxxxxxxx> wrote:
>
> The upstream changes to dropck predate the PR I linked up-thread which
> landed in 2017. Since this Arc code was written in 2022, it never had
> any effect. Isn't it proper to keep the "Fixes" tag?
If there is a bug, definitely yes, but I don't think that applies is
-- this is more of a cleanup, no?
Sometimes things are marked as "fixes" that are perhaps a stretch
(e.g. a typo in a comment). It depends a bit on the maintainer and how
we define "bug" (e.g. does it count in docs, or just actual end
users). But here it just seems something is superfluous, at worst, and
it does not need to be backported either. Even if we kept the tag for
some reason, I think this belongs in `rust-next`.
But if I am missing something, and this does indeed fix something that
we should prioritize, please let me know!
What looks more important, to me, is to clarify/document why (or why
not) we have it, regardless of whether we keep it or not, i.e. having
thought about it, I think it wouldn't hurt having a line/comment even
if we remove it.
Thanks!
Cheers,
Miguel