Remove RefCell<Option> from PARKER

[taiki-e]

No description provided.

[taiki-e]

Nah, it is better to initialize the parker in the TLS initialization code than to do this.

[taiki-e]

Ok, removed both RefCell and Option from it.

[taiki-e]

event-listener/src/lib.rs

Lines 1060 to 1062 in b1d437a

	let mut pair = parker
	.try_borrow_mut()
	.expect("Shouldn't be able to borrow parker reentrantly");

Is RefCell used to detect reentrant parking? This seems true in other places (futures-lite, async-io) where we need to wait for arbitrary futures, but that should not happen in event-listener, right?

• https://github.com/smol-rs/futures-lite/blob/2e555681d54dedb5f18455deb8d403386a368729/src/future.rs#L89
• https://github.com/smol-rs/async-io/blob/1c635937427c70300118bd8718b4c6c5c8645da8/src/driver.rs#L186

[notgull]

Thanks!

[notgull]

Is RefCell used to detect reentrant parking? This seems true in other places (futures-lite, async-io) where we need to wait for arbitrary futures, but that should not happen in event-listener, right?

This is right, since you can't run arbitrary code while parked.