On Mon May 18, 2026 at 5:36 PM CEST, Gary Guo wrote:
>> +    fn probe<'bound>(
>> +        pdev: &'bound pci::Device<Core>,
>
> This is technically incorrect, as the `Device<Core>` only lives as long as the
> callback. Having `'bound` would be incorrect.

I think that's more on the theoretical side, as Core is not Sync, but I agree it
makes sense to address.

> or perhaps by adding a lifetime to `Core<'_>`
>
>     pdev: &'bound pci::Device<Core<'_>>,

This seems reasonable.

Reply via email to