Re: [PATCH v3 3/4] rust: devres: get rid of Devres' inner Arc

From: Danilo Krummrich
Date: Thu Jun 26 2025 - 15:47:36 EST


On Wed, Jun 25, 2025 at 09:13:24PM -0700, Boqun Feng wrote:
> On Tue, Jun 24, 2025 at 11:54:01PM +0200, Danilo Krummrich wrote:
> `T` also need to be `Send` for `Devres<T>` to be `Sync` because that's
> what `Revocable<T>` requires.

I think that's because of Revocable::revoke(), however you can't call
Devres::revoke().

> (Unless we want `T` always being `Send`
> because of the issue I mentioned above)

Yes, we do.

> The rest looks good to me.

Great! :)