Use the OR operator and clarify the calculation of fractionLostAndPac…

…ketsLost Co-authored-by: Paul-Louis Ageneau <[email protected]>
paullouisageneau · Dec 11, 2023 · 45ff323 · 45ff323
1 parent d23ad4b
commit 45ff323
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/src/rtp.cpp b/src/rtp.cpp
@@ -171,7 +171,7 @@ void RtcpReportBlock::setSSRC(SSRC in_ssrc) { _ssrc = htonl(in_ssrc); }
 
 void RtcpReportBlock::setPacketsLost(uint8_t fractionLost,
                                      unsigned int packetsLostCount) {
-	_fractionLostAndPacketsLost = ((uint32_t)fractionLost << 24) & htonl(packetsLostCount);
+	_fractionLostAndPacketsLost = htonl((uint32_t(fractionLost) << 24) | (packetsLostCount & 0xFFFFFF));
 }
 
 uint8_t RtcpReportBlock::getFractionLost() const {