First off, this is a very nice and simple pinging crate, so thank you for writing this. However, it would be nice if the functions add_ipaddr and remove_ipaddr returned a result of their IP parsing instead of handling it on their own. Them parsing it on their own can result in the pinger running indefinitely when a wrong IP is specified and therefore no IPs are present.
First off, this is a very nice and simple pinging crate, so thank you for writing this. However, it would be nice if the functions
add_ipaddrandremove_ipaddrreturned a result of their IP parsing instead of handling it on their own. Them parsing it on their own can result in the pinger running indefinitely when a wrong IP is specified and therefore no IPs are present.