Remove unnecessary casts

Don't need to cast nil to a pointer
pion · Apr 16, 2024 · c19dc46 · c19dc46
1 parent ace759a
commit c19dc46
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/pkg/jitterbuffer/jitter_buffer.go b/pkg/jitterbuffer/jitter_buffer.go
@@ -213,7 +213,7 @@ func (jb *JitterBuffer) Pop() (*rtp.Packet, error) {
 	if err != nil {
 		jb.stats.underflowCount++
 		jb.emit(BufferUnderflow)
-		return (*rtp.Packet)(nil), err
+		return nil, err
 	}
 	jb.playoutHead = (jb.playoutHead + 1) % math.MaxUint16
 	jb.updateState()
@@ -231,7 +231,7 @@ func (jb *JitterBuffer) PopAtSequence(sq uint16) (*rtp.Packet, error) {
 	if err != nil {
 		jb.stats.underflowCount++
 		jb.emit(BufferUnderflow)
-		return (*rtp.Packet)(nil), err
+		return nil, err
 	}
 	jb.playoutHead = (jb.playoutHead + 1) % math.MaxUint16
 	jb.updateState()
@@ -245,7 +245,7 @@ func (jb *JitterBuffer) PeekAtSequence(sq uint16) (*rtp.Packet, error) {
 	defer jb.mutex.Unlock()
 	packet, err := jb.packets.Find(sq)
 	if err != nil {
-		return (*rtp.Packet)(nil), err
+		return nil, err
 	}
 	return packet, nil
 }
@@ -262,7 +262,7 @@ func (jb *JitterBuffer) PopAtTimestamp(ts uint32) (*rtp.Packet, error) {
 	if err != nil {
 		jb.stats.underflowCount++
 		jb.emit(BufferUnderflow)
-		return (*rtp.Packet)(nil), err
+		return nil, err
 	}
 	jb.updateState()
 	return packet, nil