Question: `shutdown_complete_rx` never used in the `Listner`

[Aklenor]

Why do you put shutdown_complete_rx: mpsc::Receiver<()> into Listner struct if it never used inside of it?

mini-redis/src/server.rs

Line 63 in ebe4e1f

shutdown_complete_rx: mpsc::Receiver<()>,

while shutdown_complete_tx: mpsc::Sender<()> seems reasonable for me.

[Darksonn]

The receiver is used here:

mini-redis/src/server.rs

Lines 203 to 207 in ebe4e1f

	// Wait for all active connections to finish processing. As the `Sender`
	// handle held by the listener has been dropped above, the only remaining
	// `Sender` instances are held by connection handler tasks. When those drop,
	// the `mpsc` channel will close and `recv()` will return `None`.
	let _ = shutdown_complete_rx.recv().await;

[Aklenor]

I see that but why to put it inside of the Listner struct, if you use it only in run function. I made PR with changes that I mean.