Call RTCPeerConnection.Ontrack with a new goroutine

Every implementation should do this anyway. Also
new users might not understand and block all events for
RTCPeerConnections

examples/gstreamer/main.go

@@ -38,16 +38,14 @@ func startWebrtc(pipeline *gst.Pipeline) {
 	// Set a handler for when a new remote track starts, this handler starts a gstreamer pipeline
 	// with the first track and assumes it is VP8 video data.
 	peerConnection.Ontrack = func(mediaType webrtc.TrackType, packets <-chan *rtp.Packet) {
-		go func() {
-			track := atomic.AddUint64(&trackCount, 1)
-			fmt.Printf("Track %d has started \n", track)
-			if track == 1 && mediaType == webrtc.VP8 {
-				for {
-					p := <-packets
-					pipeline.Push(p.Raw)
-				}
+		track := atomic.AddUint64(&trackCount, 1)
+		fmt.Printf("Track %d has started \n", track)
+		if track == 1 && mediaType == webrtc.VP8 {
+			for {
+				p := <-packets
+				pipeline.Push(p.Raw)
 			}
-		}()
+		}
 	}
 
 	// Set the remote SessionDescription

examples/save-to-disk/main.go

@@ -37,18 +37,16 @@ func main() {
 	// an ivf file, since we could have multiple video tracks we provide a counter.
 	// In your application this is where you would handle/process video
 	peerConnection.Ontrack = func(mediaType webrtc.TrackType, packets <-chan *rtp.Packet) {
-		go func() {
-			track := atomic.AddUint64(&trackCount, 1)
-			fmt.Printf("Track %d has started \n", track)
+		track := atomic.AddUint64(&trackCount, 1)
+		fmt.Printf("Track %d has started \n", track)
 
-			i, err := newIVFWriter(fmt.Sprintf("output-%d.ivf", track))
-			if err != nil {
-				panic(err)
-			}
-			for {
-				i.addPacket(<-packets)
-			}
-		}()
+		i, err := newIVFWriter(fmt.Sprintf("output-%d.ivf", track))
+		if err != nil {
+			panic(err)
+		}
+		for {
+			i.addPacket(<-packets)
+		}
 	}
 
 	// Set the remote SessionDescription

rtcpeerconnection.go

@@ -92,6 +92,6 @@ func (r *RTCPeerConnection) generateChannel(ssrc uint32) (buffers chan<- *rtp.Pa
 	}
 
 	bufferTransport := make(chan *rtp.Packet, 15)
-	r.Ontrack(VP8, bufferTransport) // TODO look up media via SSRC in remote SD
+	go r.Ontrack(VP8, bufferTransport) // TODO look up media via SSRC in remote SD
 	return bufferTransport
 }