From 80b751e95e9a545d965ef230d68ca7e80a497ecf Mon Sep 17 00:00:00 2001 From: Tommi Date: Thu, 21 Dec 2023 14:08:26 +0100 Subject: Update LegacyStatsCollector to conform with Wc++11-narrowing Cherry-picked from upstream: https://webrtc-review.googlesource.com/c/src/+/332240 Bug: 322868540 Change-Id: Ida6a1af5c324473a55ea4f3b143862ea016ff50a --- pc/legacy_stats_collector.cc | 51 +++++++++++++++++++++++++++----------------- 1 file changed, 32 insertions(+), 19 deletions(-) diff --git a/pc/legacy_stats_collector.cc b/pc/legacy_stats_collector.cc index b710bc16e6..53b1d47563 100644 --- a/pc/legacy_stats_collector.cc +++ b/pc/legacy_stats_collector.cc @@ -187,9 +187,10 @@ void ExtractStats(const cricket::VoiceReceiverInfo& info, {StatsReport::kStatsValueNameAccelerateRate, info.accelerate_rate}, {StatsReport::kStatsValueNamePreemptiveExpandRate, info.preemptive_expand_rate}, - {StatsReport::kStatsValueNameTotalAudioEnergy, info.total_output_energy}, + {StatsReport::kStatsValueNameTotalAudioEnergy, + static_cast(info.total_output_energy)}, {StatsReport::kStatsValueNameTotalSamplesDuration, - info.total_output_duration}}; + static_cast(info.total_output_duration)}}; const IntForAdd ints[] = { {StatsReport::kStatsValueNameCurrentDelayMs, info.delay_estimate_ms}, @@ -243,9 +244,10 @@ void ExtractStats(const cricket::VoiceSenderInfo& info, SetAudioProcessingStats(report, info.apm_statistics); const FloatForAdd floats[] = { - {StatsReport::kStatsValueNameTotalAudioEnergy, info.total_input_energy}, + {StatsReport::kStatsValueNameTotalAudioEnergy, + static_cast(info.total_input_energy)}, {StatsReport::kStatsValueNameTotalSamplesDuration, - info.total_input_duration}}; + static_cast(info.total_input_duration)}}; RTC_DCHECK_GE(info.audio_level, 0); const IntForAdd ints[] = { @@ -339,7 +341,8 @@ void ExtractStats(const cricket::VideoReceiverInfo& info, {StatsReport::kStatsValueNamePlisSent, info.plis_sent}, {StatsReport::kStatsValueNameRenderDelayMs, info.render_delay_ms}, {StatsReport::kStatsValueNameTargetDelayMs, info.target_delay_ms}, - {StatsReport::kStatsValueNameFramesDecoded, info.frames_decoded}, + {StatsReport::kStatsValueNameFramesDecoded, + static_cast(info.frames_decoded)}, }; for (const auto& i : ints) @@ -383,15 +386,19 @@ void ExtractStats(const cricket::VideoSenderInfo& info, info.encode_usage_percent}, {StatsReport::kStatsValueNameFirsReceived, info.firs_rcvd}, {StatsReport::kStatsValueNameFrameHeightSent, info.send_frame_height}, - {StatsReport::kStatsValueNameFrameRateInput, round(info.framerate_input)}, + {StatsReport::kStatsValueNameFrameRateInput, + static_cast(round(info.framerate_input))}, {StatsReport::kStatsValueNameFrameRateSent, info.framerate_sent}, {StatsReport::kStatsValueNameFrameWidthSent, info.send_frame_width}, - {StatsReport::kStatsValueNameNacksReceived, info.nacks_rcvd}, + {StatsReport::kStatsValueNameNacksReceived, + static_cast(info.nacks_rcvd)}, {StatsReport::kStatsValueNamePacketsLost, info.packets_lost}, {StatsReport::kStatsValueNamePacketsSent, info.packets_sent}, {StatsReport::kStatsValueNamePlisReceived, info.plis_rcvd}, - {StatsReport::kStatsValueNameFramesEncoded, info.frames_encoded}, - {StatsReport::kStatsValueNameHugeFramesSent, info.huge_frames_sent}, + {StatsReport::kStatsValueNameFramesEncoded, + static_cast(info.frames_encoded)}, + {StatsReport::kStatsValueNameHugeFramesSent, + static_cast(info.huge_frames_sent)}, }; for (const auto& i : ints) @@ -780,19 +787,25 @@ StatsReport* LegacyStatsCollector::AddConnectionInfoReport( AddCandidateReport(remote_candidate_stats, false)->id()); const Int64ForAdd int64s[] = { - {StatsReport::kStatsValueNameBytesReceived, info.recv_total_bytes}, - {StatsReport::kStatsValueNameBytesSent, info.sent_total_bytes}, - {StatsReport::kStatsValueNamePacketsSent, info.sent_total_packets}, - {StatsReport::kStatsValueNameRtt, info.rtt}, + {StatsReport::kStatsValueNameBytesReceived, + static_cast(info.recv_total_bytes)}, + {StatsReport::kStatsValueNameBytesSent, + static_cast(info.sent_total_bytes)}, + {StatsReport::kStatsValueNamePacketsSent, + static_cast(info.sent_total_packets)}, + {StatsReport::kStatsValueNameRtt, static_cast(info.rtt)}, {StatsReport::kStatsValueNameSendPacketsDiscarded, - info.sent_discarded_packets}, + static_cast(info.sent_discarded_packets)}, {StatsReport::kStatsValueNameSentPingRequestsTotal, - info.sent_ping_requests_total}, + static_cast(info.sent_ping_requests_total)}, {StatsReport::kStatsValueNameSentPingRequestsBeforeFirstResponse, - info.sent_ping_requests_before_first_response}, - {StatsReport::kStatsValueNameSentPingResponses, info.sent_ping_responses}, - {StatsReport::kStatsValueNameRecvPingRequests, info.recv_ping_requests}, - {StatsReport::kStatsValueNameRecvPingResponses, info.recv_ping_responses}, + static_cast(info.sent_ping_requests_before_first_response)}, + {StatsReport::kStatsValueNameSentPingResponses, + static_cast(info.sent_ping_responses)}, + {StatsReport::kStatsValueNameRecvPingRequests, + static_cast(info.recv_ping_requests)}, + {StatsReport::kStatsValueNameRecvPingResponses, + static_cast(info.recv_ping_responses)}, }; for (const auto& i : int64s) report->AddInt64(i.name, i.value); -- cgit v1.2.3