mediasoup 3.20.0 → 3.20.2

package/worker/test/src/RTC/ICE/TestStunPacket.cpp

@@ -98,7 +98,7 @@ SCENARIO("ICE StunPacket", "[serializable][ice][stunpacket]")
 		  request->CheckAuthentication(usernameFragment1, password) ==
 		  RTC::ICE::StunPacket::AuthenticationResult::OK);
 
-		// Trying to modify a STUN Packet once protected must throw.
+		// Trying to modify a STUN packet once protected must throw.
 		REQUIRE_THROWS_AS(request->Protect("qweqwe"), MediaSoupError);
 		REQUIRE_THROWS_AS(request->Protect(), MediaSoupError);
 
@@ -138,7 +138,7 @@ SCENARIO("ICE StunPacket", "[serializable][ice][stunpacket]")
 		  request->CheckAuthentication(usernameFragment1, password) ==
 		  RTC::ICE::StunPacket::AuthenticationResult::OK);
 
-		// Trying to modify a STUN Packet once protected must throw.
+		// Trying to modify a STUN packet once protected must throw.
 		REQUIRE_THROWS_AS(request->Protect("qweqwe"), MediaSoupError);
 		REQUIRE_THROWS_AS(request->Protect(), MediaSoupError);
 
@@ -178,7 +178,7 @@ SCENARIO("ICE StunPacket", "[serializable][ice][stunpacket]")
 		  request->CheckAuthentication(usernameFragment1, password) ==
 		  RTC::ICE::StunPacket::AuthenticationResult::OK);
 
-		// Trying to modify a STUN Packet once protected must throw.
+		// Trying to modify a STUN packet once protected must throw.
 		REQUIRE_THROWS_AS(request->Protect("qweqwe"), MediaSoupError);
 		REQUIRE_THROWS_AS(request->Protect(), MediaSoupError);
 	}
@@ -516,7 +516,7 @@ SCENARIO("ICE StunPacket", "[serializable][ice][stunpacket]")
 		uint16_t errorCode            = 666;
 		std::string errorReasonPhrase = "UPPS UNKNOWN ERROR 😊";
 
-		// Total length of the Attributes.
+		// Total length of the attributes.
 		size_t attributesLen =
 		  (4 + 27 + 1) + (4 + 4) + (4 + 8) + (4) + (4 + 4) + (4 + 18 + 2) + (4 + 4 + 23 + 1);
 
@@ -528,7 +528,7 @@ SCENARIO("ICE StunPacket", "[serializable][ice][stunpacket]")
 		request->AddSoftware(software);
 		request->AddErrorCode(errorCode, errorReasonPhrase);
 
-		// It should fail if we try to add a duplicated Attribute.
+		// It should fail if we try to add a duplicated attribute.
 		REQUIRE_THROWS_AS(request->AddUsername(username), MediaSoupError);
 		REQUIRE_THROWS_AS(request->AddPriority(priority), MediaSoupError);
 		REQUIRE_THROWS_AS(request->AddIceControlling(iceControlling), MediaSoupError);
@@ -646,7 +646,7 @@ SCENARIO("ICE StunPacket", "[serializable][ice][stunpacket]")
 		    transactionId,
 		    RTC::ICE::StunPacket::TransactionIdLength));
 
-		/* Protect the STUN Packet. */
+		/* Protect the STUN packet. */
 
 		std::string password = "asjhdkjhkasd";
 
@@ -683,7 +683,7 @@ SCENARIO("ICE StunPacket", "[serializable][ice][stunpacket]")
 		  request->CheckAuthentication(usernameFragment1, password) ==
 		  RTC::ICE::StunPacket::AuthenticationResult::OK);
 
-		// Trying to modify a STUN Packet once protected must throw.
+		// Trying to modify a STUN packet once protected must throw.
 		REQUIRE_THROWS_AS(request->Protect("qweqwe"), MediaSoupError);
 		REQUIRE_THROWS_AS(request->Protect(), MediaSoupError);
 	}
@@ -732,7 +732,7 @@ SCENARIO("ICE StunPacket", "[serializable][ice][stunpacket]")
 
 		uv_ip4_addr("22.33.0.125", 5678, xorMappedAddressIn);
 
-		// Total length of the Attributes.
+		// Total length of the attributes.
 		size_t attributesLen = (4 + 8);
 
 		successResponse->AddXorMappedAddress(xorMappedAddress);
@@ -797,7 +797,7 @@ SCENARIO("ICE StunPacket", "[serializable][ice][stunpacket]")
 
 		uv_ip6_addr("2001:db8::1234", 20002, xorMappedAddressIn6);
 
-		// Total length of the Attributes.
+		// Total length of the attributes.
 		attributesLen = (4 + 20);
 
 		successResponse->AddXorMappedAddress(xorMappedAddress);
@@ -838,7 +838,7 @@ SCENARIO("ICE StunPacket", "[serializable][ice][stunpacket]")
 		REQUIRE(ip == Utils::IP::NormalizeIp(expectedIp));
 		REQUIRE(port == 20002);
 
-		/* Protect the STUN Packet. */
+		/* Protect the STUN packet. */
 
 		std::string password = "asjhdkjhkasd";
 
@@ -875,7 +875,7 @@ SCENARIO("ICE StunPacket", "[serializable][ice][stunpacket]")
 		  successResponse->CheckAuthentication(password) ==
 		  RTC::ICE::StunPacket::AuthenticationResult::OK);
 
-		// Trying to modify a STUN Packet once protected must throw.
+		// Trying to modify a STUN packet once protected must throw.
 		REQUIRE_THROWS_AS(successResponse->Protect("qweqwe"), MediaSoupError);
 		REQUIRE_THROWS_AS(successResponse->Protect(), MediaSoupError);
 	}
@@ -918,7 +918,7 @@ SCENARIO("ICE StunPacket", "[serializable][ice][stunpacket]")
 		uint16_t errorCode            = 666;
 		std::string errorReasonPhrase = "UPPS UNKNOWN ERROR 😊";
 
-		// Total length of the Attributes.
+		// Total length of the attributes.
 		size_t attributesLen = (4 + 4 + 23 + 1);
 
 		errorResponse->AddErrorCode(errorCode, errorReasonPhrase);
@@ -949,7 +949,7 @@ SCENARIO("ICE StunPacket", "[serializable][ice][stunpacket]")
 		                  /*hasMessageIntegrity*/ false,
 		                  /*hasFingerprint*/ false);
 
-		/* Protect the STUN Packet. */
+		/* Protect the STUN packet. */
 
 		std::string password = "23786asdas123";
 
@@ -985,7 +985,7 @@ SCENARIO("ICE StunPacket", "[serializable][ice][stunpacket]")
 		REQUIRE(
 		  errorResponse->CheckAuthentication(password) == RTC::ICE::StunPacket::AuthenticationResult::OK);
 
-		// Trying to modify a STUN Packet once protected must throw.
+		// Trying to modify a STUN packet once protected must throw.
 		REQUIRE_THROWS_AS(errorResponse->Protect("qweqwe"), MediaSoupError);
 		REQUIRE_THROWS_AS(errorResponse->Protect(), MediaSoupError);
 
@@ -1002,7 +1002,7 @@ SCENARIO("ICE StunPacket", "[serializable][ice][stunpacket]")
 		errorCode         = 400;
 		errorReasonPhrase = "Bad Request";
 
-		// Total length of the Attributes.
+		// Total length of the attributes.
 		attributesLen = (4 + 4 + 11 + 1);
 
 		errorResponse->AddErrorCode(errorCode, errorReasonPhrase);
@@ -1033,9 +1033,9 @@ SCENARIO("ICE StunPacket", "[serializable][ice][stunpacket]")
 		                  /*hasMessageIntegrity*/ false,
 		                  /*hasFingerprint*/ false);
 
-		/* Protect the STUN Packet (without password). */
+		/* Protect the STUN packet (without password). */
 
-		// Protect() without password only adds FINGERPRINT Attribute.
+		// Protect() without password only adds FINGERPRINT attribute.
 		errorResponse->Protect();
 
 		CHECK_STUN_PACKET(/*packet*/ errorResponse.get(),
@@ -1064,12 +1064,12 @@ SCENARIO("ICE StunPacket", "[serializable][ice][stunpacket]")
 		                  /*hasMessageIntegrity*/ false,
 		                  /*hasFingerprint*/ true);
 
-		// Cannot check authentication in a STUN Packet without MESSAGE-INTEGRITY.
+		// Cannot check authentication in a STUN packet without MESSAGE-INTEGRITY.
 		REQUIRE(
 		  errorResponse->CheckAuthentication(password) ==
 		  RTC::ICE::StunPacket::AuthenticationResult::BAD_MESSAGE);
 
-		// Trying to modify a STUN Packet once protected must throw.
+		// Trying to modify a STUN packet once protected must throw.
 		REQUIRE_THROWS_AS(errorResponse->Protect("qweqwe"), MediaSoupError);
 		REQUIRE_THROWS_AS(errorResponse->Protect(), MediaSoupError);
 	}
@@ -1151,7 +1151,7 @@ SCENARIO("ICE StunPacket", "[serializable][ice][stunpacket]")
 		  successResponse->CheckAuthentication("qwekqjhwekjahsd") ==
 		  RTC::ICE::StunPacket::AuthenticationResult::OK);
 
-		// Trying to modify a STUN Packet once protected must throw.
+		// Trying to modify a STUN packet once protected must throw.
 		REQUIRE_THROWS_AS(successResponse->Protect("qweqwe"), MediaSoupError);
 		REQUIRE_THROWS_AS(successResponse->Protect(), MediaSoupError);
 	}
@@ -1167,7 +1167,7 @@ SCENARIO("ICE StunPacket", "[serializable][ice][stunpacket]")
 		std::unique_ptr<RTC::ICE::StunPacket> errorResponse{ request->CreateErrorResponse(
 			iceCommon::ResponseFactoryBuffer, sizeof(iceCommon::ResponseFactoryBuffer), 666, "BAD STUFF") };
 
-		// Total length of the Attributes (ERROR-CODE).
+		// Total length of the attributes (ERROR-CODE).
 		const size_t attributesLen = (4 + 4 + 9 + 3);
 
 		CHECK_STUN_PACKET(/*packet*/ errorResponse.get(),
@@ -1236,7 +1236,7 @@ SCENARIO("ICE StunPacket", "[serializable][ice][stunpacket]")
 		  errorResponse->CheckAuthentication("qwekqjhwekjahsd") ==
 		  RTC::ICE::StunPacket::AuthenticationResult::OK);
 
-		// Trying to modify a STUN Packet once protected must throw.
+		// Trying to modify a STUN packet once protected must throw.
 		REQUIRE_THROWS_AS(errorResponse->Protect("qweqwe"), MediaSoupError);
 		REQUIRE_THROWS_AS(errorResponse->Protect(), MediaSoupError);
 	}

package/worker/test/src/RTC/ICE/iceCommon.cpp

@@ -3,7 +3,7 @@
 
 namespace iceCommon
 {
-	// NOTE: We don't need `alignas(4)` for STUN Packet parsing. However we do it
+	// NOTE: We don't need `alignas(4)` for STUN packet parsing. However we do it
 	// for consistency with rtpCommon.cpp and sctpCommon.cpp.
 	alignas(4) thread_local uint8_t FactoryBuffer[];
 	alignas(4) thread_local uint8_t ResponseFactoryBuffer[];

package/worker/test/src/RTC/RTCP/TestFeedbackRtpEcn.cpp

@@ -8,16 +8,16 @@ SCENARIO("RTCP Feedback RTP ECN", "[rtcp][feedback-rtp][ecn]")
 	// clang-format off
 	alignas(4) uint8_t buffer[] =
 	{
-		0x88, 0xcd, 0x00, 0x07, // Type: 205 (Generic RTP Feedback), Count: 8 (ECN) Length: 7
+		0x88, 0xcd, 0x00, 0x07, // Type: 205 (Generic RTP feedback), Count: 8 (ECN) Length: 7
 		0x00, 0x00, 0x00, 0x01, // Sender SSRC: 0x00000001
 		0x03, 0x30, 0xbd, 0xee, // Media source SSRC: 0x0330bdee
-		0x00, 0x00, 0x00, 0x01, // Extended Highest Sequence Number
-		0x00, 0x00, 0x00, 0x01, // ECT (0) Counter
-		0x00, 0x00, 0x00, 0x01, // ECT (1) Counter
-		0x00, 0x01,             // ECN-CE Counter
-		            0x00, 0x01, // not-ECT Counter
-		0x00, 0x01,             // Lost Packets Counter
-		            0x00, 0x01  // Duplication Counter
+		0x00, 0x00, 0x00, 0x01, // Extended highest sequence number
+		0x00, 0x00, 0x00, 0x01, // ECT (0) counter
+		0x00, 0x00, 0x00, 0x01, // ECT (1) counter
+		0x00, 0x01,             // ECN-CE counter
+		            0x00, 0x01, // not-ECT counter
+		0x00, 0x01,             // Lost Packets counter
+		            0x00, 0x01  // Duplication counter
 	};
 	// clang-format on
 

package/worker/test/src/RTC/RTCP/TestFeedbackRtpNack.cpp

@@ -10,7 +10,7 @@ SCENARIO("RTCP Feedback RTP NACK", "[rtcp][feedback-rtp][nack]")
 	// clang-format off
 	alignas(4) uint8_t buffer[] =
 	{
-		0x81, 0xcd, 0x00, 0x03, // Type: 205 (Generic RTP Feedback), Length: 3
+		0x81, 0xcd, 0x00, 0x03, // Type: 205 (Generic RTP feedback), Length: 3
 		0x00, 0x00, 0x00, 0x01, // Sender SSRC: 0x00000001
 		0x03, 0x30, 0xbd, 0xee, // Media source SSRC: 0x0330bdee
 		0x0b, 0x8f, 0x00, 0x03  // NACK PID: 2959, NACK BLP: 0x0003

package/worker/test/src/RTC/RTCP/TestFeedbackRtpSrReq.cpp

@@ -10,7 +10,7 @@ SCENARIO("RTCP Feedback RTP SR-REQ", "[rtcp][feedback-rtp][sr-req]")
 	// clang-format off
 	alignas(4) uint8_t buffer[] =
 	{
-		0x85, 0xcd, 0x00, 0x02, // Type: 205 (Generic RTP Feedback), Count: 5 (SR-REQ) Length: 3
+		0x85, 0xcd, 0x00, 0x02, // Type: 205 (Generic RTP feedback), Count: 5 (SR-REQ) Length: 3
 		0x00, 0x00, 0x00, 0x01, // Sender SSRC: 0x00000001
 		0x03, 0x30, 0xbd, 0xee, // Media source SSRC: 0x0330bdee
 	};

package/worker/test/src/RTC/RTCP/TestFeedbackRtpTllei.cpp

@@ -10,7 +10,7 @@ SCENARIO("RTCP Feedback RTP TLLEI", "[rtcp][feedback-rtp][tllei]")
 	// clang-format off
 	alignas(4) uint8_t buffer[] =
 	{
-		0x87, 0xcd, 0x00, 0x03, // Type: 205 (Generic RTP Feedback), Count: 7 (TLLEI) Length: 3
+		0x87, 0xcd, 0x00, 0x03, // Type: 205 (Generic RTP feedback), Count: 7 (TLLEI) Length: 3
 		0x00, 0x00, 0x00, 0x01, // Sender SSRC: 0x00000001
 		0x03, 0x30, 0xbd, 0xee, // Media source SSRC: 0x0330bdee
 		0x00, 0x01, 0xaa, 0x55

package/worker/test/src/RTC/RTCP/TestFeedbackRtpTmmb.cpp

@@ -10,7 +10,7 @@ SCENARIO("RTCP Feedback RTP TMMBR", "[rtcp][feedback-rtp][tmmb]")
 	// clang-format off
 	alignas(4) uint8_t buffer[] =
 	{
-		0x83, 0xcd, 0x00, 0x04, // Type: 205 (Generic RTP Feedback), Count: 8 (TMMBR) Length: 7
+		0x83, 0xcd, 0x00, 0x04, // Type: 205 (Generic RTP feedback), Count: 8 (TMMBR) Length: 7
 		0x00, 0x00, 0x00, 0x01, // Sender SSRC: 0x00000001
 		0x03, 0x30, 0xbd, 0xee, // Media source SSRC: 0x0330bdee
 		0x02, 0xd0, 0x37, 0x02, // SSRC: 0x02d03702

package/worker/test/src/RTC/RTCP/TestSenderReport.cpp

@@ -5,7 +5,7 @@
 
 SCENARIO("RTCP SenderReport", "[rtcp][sender-report]")
 {
-	// RTCP Packet. Sender Report and Receiver Report.
+	// RTCP packet. Sender Report and Receiver Report.
 
 	// clang-format off
 	alignas(4) uint8_t buffer[] =

package/worker/test/src/RTC/RTP/TestPacket.cpp

@@ -489,7 +489,7 @@ SCENARIO("RTP Packet", "[serializable][rtp][packet]")
 			0x90, 0x01, 0x00, 0x08,
 			0x00, 0x00, 0x00, 0x04,
 			0x00, 0x00, 0x00, 0x05,
-			0xbe, 0xde, 0x00, 0x03, // Header Extension
+			0xbe, 0xde, 0x00, 0x03, // Header extension
 			0x10, 0xaa, 0x21, 0xbb, // - id: 1, len: 1
 			0xff, 0x00, 0x00, 0x33, // - id: 2, len: 2
 			0xff, 0xff, 0xff, 0xff, // - id: 3, len: 4
@@ -672,7 +672,7 @@ SCENARIO("RTP Packet", "[serializable][rtp][packet]")
 			0x90, 0x01, 0x00, 0x08,
 			0x00, 0x00, 0x00, 0x04,
 			0x00, 0x00, 0x00, 0x05,
-			0x10, 0x00, 0x00, 0x04, // Header Extension
+			0x10, 0x00, 0x00, 0x04, // Header extension
 			0x00, 0x00, 0x01, 0x00, // - id: 1, len: 0
 			0x02, 0x01, 0x42, 0x00, // - id: 2, len: 1
 			0x03, 0x02, 0x11, 0x22, // - id: 3, len: 2
@@ -1016,7 +1016,7 @@ SCENARIO("RTP Packet", "[serializable][rtp][packet]")
 			0x90, 0x01, 0x00, 0x08,
 			0x00, 0x00, 0x00, 0x04,
 			0x00, 0x00, 0x00, 0x05,
-			0xbe, 0xde, 0x00, 0x03, // Header Extension
+			0xbe, 0xde, 0x00, 0x03, // Header extension
 			0x10, 0xaa, 0x21, 0xbb, // - id: 1, len: 1
 			0xff, 0x00, 0x00, 0x33, // - id: 2, len: 2
 			0xff, 0xff, 0xff, 0xff, // - id: 3, len: 4
@@ -1070,13 +1070,13 @@ SCENARIO("RTP Packet", "[serializable][rtp][packet]")
 
 		// Extensions:
 		//
-		// Using One-Byte Extensions:
-		// - Header Extension value length: 1 + 1 + 1 + 2 + 1 + 3 = 9 => 12 (padded)
-		// - Header Extension length: 4 + 12 = 16
+		// Using One-Byte extensions:
+		// - Header extension value length: 1 + 1 + 1 + 2 + 1 + 3 = 9 => 12 (padded)
+		// - Header extension length: 4 + 12 = 16
 		//
-		// Using Two-Bytes Extensions:
-		// - Header Extension value length: 2 + 1 + 2 + 2 + 2 + 3 = 12
-		// - Header Extension length: 4 + 12 = 16
+		// Using Two-Bytes extensions:
+		// - Header extension value length: 2 + 1 + 2 + 2 + 2 + 3 = 12
+		// - Header extension length: 4 + 12 = 16
 		//
 		// Extension id 1.
 		rtpCommon::DataBuffer[0] = 11;
@@ -1106,7 +1106,7 @@ SCENARIO("RTP Packet", "[serializable][rtp][packet]")
 		  /*len*/ 3,
 		  /*value*/ rtpCommon::DataBuffer + 3);
 
-		// Add One-Byte Extensions.
+		// Add One-Byte extensions.
 		packet->SetExtensions(RTC::RTP::Packet::ExtensionsType::OneByte, extensions);
 
 		packet->SetPayload(rtpCommon::DataBuffer, 10);
@@ -1349,7 +1349,7 @@ SCENARIO("RTP Packet", "[serializable][rtp][packet]")
 		    packet->GetPayload(), packet->GetPayloadLength(), rtpCommon::DataBuffer, 1) == true);
 		REQUIRE(packet->IsPaddedTo4Bytes() == true);
 
-		/* Remove Header Extension. */
+		/* Remove header extension. */
 
 		packet->RemoveHeaderExtension();
 
@@ -1383,7 +1383,7 @@ SCENARIO("RTP Packet", "[serializable][rtp][packet]")
 		    packet->GetPayload(), packet->GetPayloadLength(), rtpCommon::DataBuffer, 1) == true);
 		REQUIRE(packet->IsPaddedTo4Bytes() == true);
 
-		// Add Two-Bytes Extensions.
+		// Add Two-Bytes extensions.
 		packet->SetExtensions(RTC::RTP::Packet::ExtensionsType::TwoBytes, extensions);
 
 		CHECK_RTP_PACKET(
@@ -1439,7 +1439,7 @@ SCENARIO("RTP Packet", "[serializable][rtp][packet]")
 
 		std::vector<RTC::RTP::Packet::Extension> extensions;
 
-		// Can fit into One-Byte type Extensions.
+		// Can fit into One-Byte type extensions.
 		extensions.assign(
 		  {
 		    { RTC::RtpHeaderExtensionUri::Type::MID,           1,  1,  rtpCommon::DataBuffer },
@@ -1448,7 +1448,7 @@ SCENARIO("RTP Packet", "[serializable][rtp][packet]")
 		packet->SetExtensions(RTC::RTP::Packet::ExtensionsType::Auto, extensions);
 		REQUIRE(packet->HasOneByteExtensions());
 
-		// Requires Two-Bytes type Extensions due to id > 14.
+		// Requires Two-Bytes type extensions due to id > 14.
 		extensions.assign(
 		  {
 		    { RTC::RtpHeaderExtensionUri::Type::ABS_SEND_TIME, 15, 2, rtpCommon::DataBuffer }
@@ -1456,7 +1456,7 @@ SCENARIO("RTP Packet", "[serializable][rtp][packet]")
 		packet->SetExtensions(RTC::RTP::Packet::ExtensionsType::Auto, extensions);
 		REQUIRE(packet->HasTwoBytesExtensions());
 
-		// Requires Two-Bytes type Extensions due to length 0.
+		// Requires Two-Bytes type extensions due to length 0.
 		extensions.assign(
 		  {
 		    { RTC::RtpHeaderExtensionUri::Type::REPAIRED_RTP_STREAM_ID, 1, 0, rtpCommon::DataBuffer }
@@ -1464,7 +1464,7 @@ SCENARIO("RTP Packet", "[serializable][rtp][packet]")
 		packet->SetExtensions(RTC::RTP::Packet::ExtensionsType::Auto, extensions);
 		REQUIRE(packet->HasTwoBytesExtensions());
 
-		// Requires Two-Bytes type Extensions due to length > 16.
+		// Requires Two-Bytes type extensions due to length > 16.
 		extensions.assign(
 		  {
 		    { RTC::RtpHeaderExtensionUri::Type::TIME_OFFSET, 1, 17, rtpCommon::DataBuffer }
@@ -1595,7 +1595,7 @@ SCENARIO("RTP Packet", "[serializable][rtp][packet]")
 		std::vector<RTC::RTP::Packet::Extension> extensions;
 		auto* d = rtpCommon::DataBuffer;
 
-		// Invalid Extension id 0.
+		// Invalid extension id 0.
 		extensions.assign(
 		  {
 		    { RTC::RtpHeaderExtensionUri::Type::MID,           0, 4, d },
@@ -1609,7 +1609,7 @@ SCENARIO("RTP Packet", "[serializable][rtp][packet]")
 		  packet->SetExtensions(RTC::RTP::Packet::ExtensionsType::TwoBytes, extensions),
 		  MediaSoupTypeError);
 
-		// Invalid Extension id > 14 in One-Byte.
+		// Invalid extension id > 14 in One-Byte.
 		extensions.assign(
 		  {
 		    { RTC::RtpHeaderExtensionUri::Type::VIDEO_ORIENTATION, 15, 2, d },
@@ -1622,7 +1622,7 @@ SCENARIO("RTP Packet", "[serializable][rtp][packet]")
 		  MediaSoupTypeError);
 		REQUIRE_NOTHROW(packet->SetExtensions(RTC::RTP::Packet::ExtensionsType::TwoBytes, extensions));
 
-		// Invalid Extension length 0 in One-Byte.
+		// Invalid extension length 0 in One-Byte.
 		extensions.assign(
 		  {
 		    { RTC::RtpHeaderExtensionUri::Type::MID,                    3, 0, d },
@@ -1636,7 +1636,7 @@ SCENARIO("RTP Packet", "[serializable][rtp][packet]")
 		  MediaSoupTypeError);
 		REQUIRE_NOTHROW(packet->SetExtensions(RTC::RTP::Packet::ExtensionsType::TwoBytes, extensions));
 
-		// Invalid Extension length > 16 in One-Byte.
+		// Invalid extension length > 16 in One-Byte.
 		extensions.assign(
 		  {
 		    { RTC::RtpHeaderExtensionUri::Type::MEDIASOUP_PACKET_ID,   3,   17, d },
@@ -1828,9 +1828,9 @@ SCENARIO("RTP Packet", "[serializable][rtp][packet]")
 
 		std::vector<RTC::RTP::Packet::Extension> extensions;
 
-		// One-Byte Extensions:
-		// - Header Extension value length: 1 + 1 + 1 + 2 + 1 + 3 = 9 => 12 (padded)
-		// - Header Extension length: 4 + 12 = 16
+		// One-Byte extensions:
+		// - Header extension value length: 1 + 1 + 1 + 2 + 1 + 3 = 9 => 12 (padded)
+		// - Header extension length: 4 + 12 = 16
 		//
 		// clang-format off
 		uint8_t extension1[] =
@@ -2180,7 +2180,7 @@ SCENARIO("RTP Packet", "[serializable][rtp][packet]")
 
 		packet->SetBufferReleasedListener(std::addressof(packetBufferReleasedListener));
 
-		// If we destroy the Packet it should invoke the listener.
+		// If we destroy the packet it should invoke the listener.
 		packet.reset(nullptr);
 
 		REQUIRE(packetBufferReleased == true);
@@ -2206,14 +2206,14 @@ SCENARIO("RTP Packet", "[serializable][rtp][packet]")
 
 		packet->SetBufferReleasedListener(std::addressof(packetBufferReleasedListener));
 
-		// If we serialize the Packet into another buffer it should invoke the
+		// If we serialize the packet into another buffer it should invoke the
 		// listener.
 		packet->Serialize(rtpCommon::SerializeBuffer, sizeof(rtpCommon::SerializeBuffer));
 
 		REQUIRE(packetBufferReleased == true);
 
 		// NOTE: We need to unset the buffer released listener because once the
-		// unique_ptr of the Packet gets out of the scope, the Packet will be
+		// unique_ptr of the packet gets out of the scope, the packet will be
 		// deallocated and will invoke the buffer released listener, which at
 		// that time is already out of the scope (it's lifetime ended) so it's
 		// been destroyed.

package/worker/test/src/RTC/RTP/TestSharedPacket.cpp

@@ -49,7 +49,7 @@ SCENARIO("RTP SharedPacket", "[rtp][sharedpacket]")
 
 	SECTION("constructor with packet and copy constructor")
 	{
-		// Create sharedPacket1 using constructor with a Packet.
+		// Create sharedPacket1 using constructor with a packet.
 		RTC::RTP::SharedPacket sharedPacket1(packetA);
 
 		REQUIRE(sharedPacket1.HasPacket());

package/worker/test/src/RTC/RTP/rtpCommon.cpp

@@ -3,7 +3,7 @@
 
 namespace rtpCommon
 {
-	// NOTE: Buffers must be 4-byte aligned since RTP Packet parsing casts them
+	// NOTE: Buffers must be 4-byte aligned since RTP packet parsing casts them
 	// to structs (e.g. FixedHeader, HeaderExtension) that require 4-byte
 	// alignment. Without this, accessing multi-byte fields would be undefined
 	// behavior on strict-alignment architectures.

package/worker/test/src/RTC/SCTP/association/TestNegotiatedCapabilities.cpp

@@ -79,11 +79,11 @@ SCENARIO("SCTP Negotiated Capabilities", "[sctp][negotiatedcapabilities]")
 		auto* remoteSupportedExtensionsParameter =
 		  remoteChunk->BuildParameterInPlace<RTC::SCTP::SupportedExtensionsParameter>();
 
-		// NOTE: Missing FORWARD_TSN, but peer announced support for it via
-		// Forward-TSN-Supported Parameter negotiation).
-		// NOTE: Missing RE_CONFIG (needed for Partial Reliability Extension
+		// NOTE: Missing FORWARD-TSN, but peer announced support for it via
+		// Forward-TSN-Supported parameter negotiation).
+		// NOTE: Missing RE-CONFIG (needed for Partial Reliability Extension
 		// negotiation).
-		// NOTE: Missing I_FORWARD_TSN (needed for Message Interleaving negotiation).
+		// NOTE: Missing I-FORWARD-TSN (needed for Message Interleaving negotiation).
 		remoteSupportedExtensionsParameter->AddChunkType(RTC::SCTP::Chunk::ChunkType::I_DATA);
 		remoteSupportedExtensionsParameter->Consolidate();
 

package/worker/test/src/RTC/SCTP/packet/TestPacket.cpp

@@ -137,8 +137,8 @@ SCENARIO("SCTP Packet", "[serializable][sctp][packet]")
 			0xEE, 0b00001100, 0x00, 0x07,
 			// Unknown data: 0xAABBCC, 1 byte of padding
 			0xAA, 0xBB, 0xCC, 0x00,
-			// Chunk 3: Type:5 (HEARTBEAT_ACK), Flags:0b00000000, Length: 10
-			// NOTE: Chunk Length field must exclude padding of the last Parameter.
+			// Chunk 3: Type:5 (HEARTBEAT-ACK), Flags:0b00000000, Length: 10
+			// NOTE: Chunk length field must exclude padding of the last parameter.
 			0x05, 0b00000000, 0x00, 0x0A,
 			// Parameter 1: Type:1 (HEARBEAT_INFO), Length: 6
 			0x00, 0x01, 0x00, 0x06,
@@ -521,7 +521,7 @@ SCENARIO("SCTP Packet", "[serializable][sctp][packet]")
 
 		REQUIRE(packet->GetFirstChunkOfType<RTC::SCTP::InitChunk>() == nullptr);
 
-		/* Modify the Packet and add Chunks. */
+		/* Modify the packet and add chunks. */
 
 		packet->SetSourcePort(1000);
 		packet->SetDestinationPort(6000);
@@ -537,7 +537,7 @@ SCENARIO("SCTP Packet", "[serializable][sctp][packet]")
 		chunk1->SetNumberOfInboundStreams(22200);
 		chunk1->SetInitialTsn(14141414);
 
-		// Parameter 1.1: IPV4_ADDRESS, length: 8 bytes.
+		// Parameter 1.1: IPV4-ADDRESS, length: 8 bytes.
 		// NOLINTNEXTLINE (readability-identifier-naming)
 		auto* parameter1_1 = chunk1->BuildParameterInPlace<RTC::SCTP::IPv4AddressParameter>();
 
@@ -549,7 +549,7 @@ SCENARIO("SCTP Packet", "[serializable][sctp][packet]")
 
 		REQUIRE(chunk1->GetFirstParameterOfType<RTC::SCTP::IPv4AddressParameter>() == parameter1_1);
 
-		// Parameter 1.2: COOKIE_PRESERVATIVE, length: 8 bytes.
+		// Parameter 1.2: COOKIE-PRESERVATIVE, length: 8 bytes.
 		// NOLINTNEXTLINE (readability-identifier-naming)
 		auto* parameter1_2 = chunk1->BuildParameterInPlace<RTC::SCTP::CookiePreservativeParameter>();
 
@@ -558,7 +558,7 @@ SCENARIO("SCTP Packet", "[serializable][sctp][packet]")
 
 		REQUIRE(chunk1->GetFirstParameterOfType<RTC::SCTP::CookiePreservativeParameter>() == parameter1_2);
 
-		// Consolidate Chunk 1 after consolidating its Parameters 1.1 and 1.2.
+		// Consolidate chunk 1 after consolidating its parameters 1.1 and 1.2.
 		chunk1->Consolidate();
 
 		REQUIRE(chunk1->GetFirstParameterOfType<RTC::SCTP::IPv4AddressParameter>() == parameter1_1);
@@ -566,10 +566,10 @@ SCENARIO("SCTP Packet", "[serializable][sctp][packet]")
 
 		REQUIRE(packet->GetFirstChunkOfType<RTC::SCTP::InitChunk>() == chunk1);
 
-		// Chunk 2: HEARTBEAT_REQUEST, length: 4 bytes.
+		// Chunk 2: HEARTBEAT-REQUEST, length: 4 bytes.
 		auto* chunk2 = packet->BuildChunkInPlace<RTC::SCTP::HeartbeatRequestChunk>();
 
-		// Parameter 2.1: HEARTBEAT_INFO, length: 4 bytes.
+		// Parameter 2.1: HEARTBEAT-INFO, length: 4 bytes.
 		// NOLINTNEXTLINE (readability-identifier-naming)
 		auto* parameter2_1 = chunk2->BuildParameterInPlace<RTC::SCTP::HeartbeatInfoParameter>();
 
@@ -581,7 +581,7 @@ SCENARIO("SCTP Packet", "[serializable][sctp][packet]")
 
 		std::memset(sctpCommon::DataBuffer, 0xFF, 3);
 
-		// Consolidate the Chunk after consolidating its Parameters.
+		// Consolidate the chunk after consolidating its parameters.
 		chunk2->Consolidate();
 
 		REQUIRE(chunk2->GetFirstParameterOfType<RTC::SCTP::HeartbeatInfoParameter>() == parameter2_1);
@@ -614,7 +614,7 @@ SCENARIO("SCTP Packet", "[serializable][sctp][packet]")
 		  /*hasValidCrc32cChecksum*/ true,
 		  /*chunksCount*/ 2);
 
-		/* Serialize the Packet. */
+		/* Serialize the packet. */
 
 		packet->Serialize(sctpCommon::SerializeBuffer, packet->GetLength());
 
@@ -635,7 +635,7 @@ SCENARIO("SCTP Packet", "[serializable][sctp][packet]")
 		REQUIRE(packet->GetFirstChunkOfType<RTC::SCTP::InitChunk>() == chunk1);
 		REQUIRE(packet->GetFirstChunkOfType<RTC::SCTP::HeartbeatRequestChunk>() == chunk2);
 
-		/* Clone the Packet. */
+		/* Clone the packet. */
 
 		packet.reset(packet->Clone(sctpCommon::CloneBuffer, packet->GetLength()));
 
@@ -759,7 +759,7 @@ SCENARIO("SCTP Packet", "[serializable][sctp][packet]")
 		packet->SetVerificationTag(3);
 		packet->SetChecksum(4);
 
-		// 4 bytes Chunk.
+		// 4 bytes chunk.
 		auto* chunk1 = RTC::SCTP::ShutdownCompleteChunk::Factory(sctpCommon::FactoryBuffer + 1000, 1000);
 
 		chunk1->SetT(true);
@@ -767,11 +767,11 @@ SCENARIO("SCTP Packet", "[serializable][sctp][packet]")
 		packet->AddChunk(chunk1);
 
 		REQUIRE(packet->GetFirstChunkOfType<RTC::SCTP::ShutdownCompleteChunk>() != nullptr);
-		// NOTE: The stored Chunk is not the same than the given one since it's
+		// NOTE: The stored chunk is not the same than the given one since it's
 		// internally cloned.
 		REQUIRE(packet->GetFirstChunkOfType<RTC::SCTP::ShutdownCompleteChunk>() != chunk1);
 
-		// Once added, we can delete the Chunk.
+		// Once added, we can delete the chunk.
 		delete chunk1;
 
 		// Packet length must be:

package/worker/test/src/RTC/SCTP/packet/chunks/TestAbortAssociationChunk.cpp

@@ -19,7 +19,7 @@ SCENARIO("SCTP Abort Association Chunk (6)", "[serializable][sctp][chunk]")
 		{
 			// Type:6 (ABORT), Flags:0b00000000, Length: 12
 			0x06, 0b00000001, 0x00, 0x0C,
-			// Error Cause 3: Code:1 (STALE_COOKIE), Length: 8
+			// Error Cause 3: Code:1 (STALE-COOKIE), Length: 8
 			0x00, 0x03, 0x00, 0x08,
 			// Measure of Staleness: 0x12345678
 			0x12, 0x34, 0x56, 0x78,
@@ -152,7 +152,7 @@ SCENARIO("SCTP Abort Association Chunk (6)", "[serializable][sctp][chunk]")
 
 		REQUIRE(chunk->GetT() == false);
 
-		/* Modify it and add Error Causes. */
+		/* Modify it and add error causes. */
 
 		chunk->SetT(true);
 
@@ -236,7 +236,7 @@ SCENARIO("SCTP Abort Association Chunk (6)", "[serializable][sctp][chunk]")
 
 		chunk->SetT(true);
 
-		// 8 bytes Error Cause.
+		// 8 bytes error cause.
 		auto* errorCause1 = RTC::SCTP::StaleCookieErrorCause::Factory(
 		  sctpCommon::FactoryBuffer + 1000, sizeof(sctpCommon::FactoryBuffer));
 
@@ -244,7 +244,7 @@ SCENARIO("SCTP Abort Association Chunk (6)", "[serializable][sctp][chunk]")
 
 		chunk->AddErrorCause(errorCause1);
 
-		// Once added, we can delete the Error Cause.
+		// Once added, we can delete the error cause.
 		delete errorCause1;
 
 		// Chunk length must be:

package/worker/test/src/RTC/SCTP/packet/chunks/TestCookieAckChunk.cpp

@@ -15,7 +15,7 @@ SCENARIO("SCTP Cookie Acknowledgement Chunk (11)", "[serializable][sctp][chunk]"
 		// clang-format off
 		alignas(4) uint8_t buffer[] =
 		{
-			// Type:11 (COOKIE_ACK), Flags:0x00000001, T: 1, Length: 4
+			// Type:11 (COOKIE-ACK), Flags:0x00000001, T: 1, Length: 4
 			0x0B, 0b00000101, 0x00, 0x04,
 			// Extra bytes that should be ignored
 			0xAA

package/worker/test/src/RTC/SCTP/packet/chunks/TestCookieEchoChunk.cpp

@@ -15,7 +15,7 @@ SCENARIO("SCTP Cookie Echo Chunk (10)", "[serializable][sctp][chunk]")
 		// clang-format off
 		alignas(4) uint8_t buffer[] =
 		{
-			// Type:A (COOKIE_ECHO), Flags: 0b00000000, Length: 9
+			// Type:A (COOKIE-ECHO), Flags: 0b00000000, Length: 9
 			0x0A, 0b00000000, 0x00, 0x09,
 			// Cookie: 0x1122334455,
 			0x11, 0x22, 0x33, 0x44,