Disable reuse of client buffers for now

Don't want buggy code in master.

Author: nhooyr

websocket.go

@@ -85,7 +85,8 @@ func (c *Conn) close(err error) {
 		if c.client {
 			go func() {
 				<-c.readLoopDone
-				// TODO this does not work if reader errors out.
+				// TODO this does not work if reader errors out so skip for now.
+				return
 				c.readDataLock <- struct{}{}
 				c.writeFrameLock <- struct{}{}
 
@@ -326,12 +327,15 @@ func (c *Conn) writePong(p []byte) error {
 }
 
 // Close closes the WebSocket connection with the given status code and reason.
+//
 // It will write a WebSocket close frame with a timeout of 5 seconds.
 // The connection can only be closed once. Additional calls to Close
 // are no-ops.
+//
 // The maximum length of reason must be 125 bytes otherwise an internal
 // error will be sent to the peer. For this reason, you should avoid
 // sending a dynamic reason.
+//
 // Close will unblock all goroutines interacting with the connection.
 func (c *Conn) Close(code StatusCode, reason string) error {
 	err := c.exportedClose(code, reason)