Fix the order of arguments of client copyLoop to match the call.

The call was copyLoop(socks, snowflake) but the function signature was func copyLoop(WebRTC, SOCKS io.ReadWriter) { The mistake was mostly harmless, because both arguments were treated the same, except that error logs would have reported the wrong direction.

Fix the order of arguments of client copyLoop to match the call.
bc5498cb · David Fifield · db1ba479 · bc5498cb
Commit bc5498cb authored Jan 28, 2020 by David Fifield
--- a/client/lib/snowflake.go
+++ b/client/lib/snowflake.go
@@ -48,7 +48,7 @@ func Handler(socks SocksConnector, snowflakes SnowflakeCollector) error {

 // Exchanges bytes between two ReadWriters.
 // (In this case, between a SOCKS and WebRTC connection.)
-func copyLoop(WebRTC, SOCKS io.ReadWriter) {
+func copyLoop(SOCKS, WebRTC io.ReadWriter) {
 	var wg sync.WaitGroup
 	wg.Add(2)
 	go func() {