Closed ghost closed 2 years ago
Looks great to me, thank you! I just fixed a small issue where the history didn't work because it created a new editor each time in the loop. If you find any more issues after the merge, you can make a new pull request, or an issue and I'll fix it.
Btw, congrats on your first pr :).
Glad you fixed the issue. Thank you so much!
On noticing repeated code patterns in the TCP and UDP handling code in
listeners.rs
, I added a decorator which takes a closure in (to perform the respective protocol operations). Some other logic like the(host, port)
check has been rewritten with slice pattern matching,if let
s andmatch
es.