Copilot review

Author: aalekseevx

pkg/flexfec/flexfec_coverage.go

@@ -150,7 +150,7 @@ func extractMask1(mask util.BitArray) uint16 {
 func extractMask2(mask util.BitArray) uint32 {
 	// We remove the first 15 bits
 	mask2 := mask.Lo << 15
-	// We get the first 31 bits (64 - 31 -> shift by 33) and we shift once more for K field
+	// We get the first 31 bits (64 - 32 -> shift by 32) and we shift once more for K field
 	mask2 >>= 33
 
 	return uint32(mask2) //nolint:gosec

pkg/flexfec/flexfec_encoder_03.go

@@ -36,7 +36,7 @@ func NewFlexEncoder03(payloadType uint8, ssrc uint32) *FlexEncoder03 {
 }
 
 // EncodeFec returns a list of generated RTP packets with FEC payloads that protect the specified mediaPackets.
-// This method return nil in case of missing RTP packets in the mediaPackets array or packets passed out of order.
+// This method returns nil in case of missing RTP packets in the mediaPackets array or packets passed out of order.
 func (flex *FlexEncoder03) EncodeFec(mediaPackets []rtp.Packet, numFecPackets uint32) []rtp.Packet {
 	// Check if mediaPackets is empty
 	if len(mediaPackets) == 0 {

pkg/flexfec/util/media_packet_iterator.go

@@ -48,5 +48,9 @@ func (m *MediaPacketIterator) Next() *rtp.Packet {
 
 // First returns the first media packet to iterate through.
 func (m *MediaPacketIterator) First() *rtp.Packet {
+	if len(m.coveredIndices) == 0 {
+		return nil
+	}
+
 	return &m.mediaPackets[m.coveredIndices[0]]
 }