From 899baa821bfcd5c46e0515c66657b8f5af71cbe5 Mon Sep 17 00:00:00 2001 From: "marpan@webrtc.org" Date: Fri, 1 Jun 2012 18:32:16 +0000 Subject: [PATCH] Temporarily disable first partition packet counting to avoid a bug in ProducerFec which doesn't properly handle important packets. TBR=stefan@webrtc.org Review URL: https://webrtc-codereview.appspot.com/631005 git-svn-id: http://webrtc.googlecode.com/svn/trunk@2346 4adac7df-926f-26a2-2b94-8c16560cd09d --- src/modules/rtp_rtcp/source/rtp_sender_video.cc | 14 +++++++++----- 1 file changed, 9 insertions(+), 5 deletions(-) diff --git a/src/modules/rtp_rtcp/source/rtp_sender_video.cc b/src/modules/rtp_rtcp/source/rtp_sender_video.cc index f1e3c746b2..a9c8be7293 100644 --- a/src/modules/rtp_rtcp/source/rtp_sender_video.cc +++ b/src/modules/rtp_rtcp/source/rtp_sender_video.cc @@ -420,11 +420,15 @@ RTPSenderVideo::SendVP8(const FrameType frameType, int packetStartPartition = packetizer.NextPacket(&dataBuffer[rtpHeaderLength], &payloadBytesInPacket, &last); - if (packetStartPartition == 0) - { - ++_numberFirstPartition; - } - else if (packetStartPartition < 0) + // TODO(holmer): Temporarily disable first partition packet counting + // to avoid a bug in ProducerFec which doesn't properly handle + // important packets. + // if (packetStartPartition == 0) + // { + // ++_numberFirstPartition; + // } + // else + if (packetStartPartition < 0) { return -1; }