fix: Remove microsecond wait in peer.(*Channel) (#1270)

kylecarbs · kylecarbs · commit 9f5c864d57dd · 2022-06-10T17:01:37.000Z
This was implemented when our WebRTC code was much less hardened. It's likely this was a cause of some other problem. Closes #1076.
diff --git a/peer/channel.go b/peer/channel.go
@@ -6,7 +6,6 @@ import (
 	"io"
 	"net"
 	"sync"
-	"time"
 
 	"github.com/pion/datachannel"
 	"github.com/pion/webrtc/v3"
@@ -245,13 +244,6 @@ func (c *Channel) Write(bytes []byte) (n int, err error) {
 	if c.dc.BufferedAmount()+uint64(len(bytes)) >= maxBufferedAmount {
 		<-c.sendMore
 	}
-	// REMARK: There's an obvious race-condition here. This is an edge-case, as
-	//         most-frequently data won't be pooled so synchronously, but is
-	//         definitely possible.
-	//
-	// See: https://github.com/pion/sctp/issues/181
-	time.Sleep(time.Microsecond)
-
 	return c.rwc.Write(bytes)
 }
 
