This PR is optional. A dedicated struct for network handling 1. makes conditions
when disconnect is called explicit and 2. keeps the network_connect method
from opening and force-updating even when a closed socket is present.
PR #24 introduces NetworkHandle, a struct wrapping TcpClient::TcpSocket to
convince the borrow checker. By chance I found it was a nice place to organize
network handling methods. This PR consists of 3 groups of commits.
Cherry-picking some of them also makes me happy.
First 2 commits are merely style changes and trivial.
The 3rd commit makes it explicit when to call disconnect during MQTT's connection establishment step.
4 - 7 divides network_connect method into 4 pieces, since it would take on too many responsibilities.
This PR is optional. A dedicated struct for network handling 1. makes conditions when
disconnectis called explicit and 2. keeps thenetwork_connectmethod from opening and force-updating even when a closed socket is present.PR #24 introduces
NetworkHandle, a struct wrappingTcpClient::TcpSocketto convince the borrow checker. By chance I found it was a nice place to organize network handling methods. This PR consists of 3 groups of commits. Cherry-picking some of them also makes me happy.disconnectduring MQTT's connection establishment step.network_connectmethod into 4 pieces, since it would take on too many responsibilities.NetworkHandle.