Refactoring

[nhooyr]

Closes #78

[codecov]

Codecov Report

Merging #81 into master will decrease coverage by 1.4%.
The diff coverage is 78.33%.

@@            Coverage Diff             @@
##           master      #81      +/-   ##
==========================================
- Coverage   82.35%   80.94%   -1.41%     
==========================================
  Files          10       10              
  Lines         731      824      +93     
==========================================
+ Hits          602      667      +65     
- Misses        108      128      +20     
- Partials       21       29       +8

Impacted Files	Coverage Δ
accept.go	88.63% <100%> (+0.4%)	⬆️
dial.go	85% <100%> (+4.35%)	⬆️
statuscode.go	100% <100%> (+5.88%)	⬆️
websocket.go	77.66% <76.44%> (-4.98%)	⬇️
statuscode_string.go	37.03% <0%> (+11.11%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3eb559b...15b8365. Read the comment docs.

[nhooyr]

This PR adds concurrent reads and massively simplifies the internals of the libraries 👏

[nhooyr]

Figured out a clever way to merge readLoop and timeoutLoop into a single goroutine!

[nhooyr]

Nvm I did not, I thought io.Reader was non blocking but it isn't. It blocks for at least a single read usually. So if we combine the two, then there is no way to get readLoop to wake up.

[nhooyr]

@coadler Definitely let me know how you feel about the additions to the API.

[coadler]

👍 will take a look tomorrow morning

[coadler]

I like the additions.

(*Conn).Context will be very helpful

[coadler]

I'll test these out on my gateway later this weekend and let you know anything else I find