GAT nature of Strategy::Future

[fogti]

event-listener-strategy/src/lib.rs

Lines 422 to 438 in 27a14c4

	pub trait Strategy<'a> {
	/// The context needed to poll the future.
	type Context: ?Sized;
	/// The future returned by the [`Strategy::wait`] method.
	type Future: Future + 'a;
	/// Poll the event listener until it is ready.
	fn poll<T>(
	&mut self,
	event_listener: Pin<&mut EventListener<T>>,
	context: &mut Self::Context,
	) -> Poll<T>;
	/// Wait for the event listener to become ready.
	fn wait(&mut self, evl: Pin<&'a mut EventListener>) -> Self::Future;
	}

We should probably keep in mind that as soon as we bump the minimum rust version to 1.65 or higher (GATs stabilization) we can probably use the following instead:

pub trait Strategy {
    /// The context needed to poll the future.
    type Context: ?Sized;

    /// The future returned by the [`Strategy::wait`] method.
    type Future<'a>: Future + 'a;

    /// Poll the event listener until it is ready.
    fn poll<T>(
        &mut self,
        event_listener: Pin<&mut EventListener<T>>,
        context: &mut Self::Context,
    ) -> Poll<T>;

    /// Wait for the event listener to become ready.
    fn wait<'a>(&mut self, evl: Pin<&'a mut EventListener>) -> Self::Future<'a>;
}

[notgull]

Yes, this is a more stable implementation of GATs. Once Debian Stable updates again we will probably be able to use actual GATs.