Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(946)

Unified Diff: webrtc/tools/event_log_visualizer/analyzer.cc

Issue 2920433002: Fix indexing error in event log analyzer. (Closed)
Patch Set: Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/tools/event_log_visualizer/analyzer.cc
diff --git a/webrtc/tools/event_log_visualizer/analyzer.cc b/webrtc/tools/event_log_visualizer/analyzer.cc
index ee41ecc527179fcc2c9ac2385f19ffe55ed2bd5e..f068759ec2a276f5b0bfff1302e2d1f0cab79273 100644
--- a/webrtc/tools/event_log_visualizer/analyzer.cc
+++ b/webrtc/tools/event_log_visualizer/analyzer.cc
@@ -345,14 +345,14 @@ EventLogAnalyzer::EventLogAnalyzer(const ParsedRtcEventLog& log)
case ParsedRtcEventLog::VIDEO_SENDER_CONFIG_EVENT: {
std::vector<rtclog::StreamConfig> configs =
parsed_log_.GetVideoSendConfig(i);
- for (size_t j = 0; j < configs.size(); j++) {
- StreamId stream(configs[i].local_ssrc, kOutgoingPacket);
+ for (const auto& config : configs) {
+ StreamId stream(config.local_ssrc, kOutgoingPacket);
extension_maps[stream] =
- RtpHeaderExtensionMap(configs[i].rtp_extensions);
+ RtpHeaderExtensionMap(config.rtp_extensions);
video_ssrcs_.insert(stream);
- StreamId rtx_stream(configs[i].rtx_ssrc, kOutgoingPacket);
+ StreamId rtx_stream(config.rtx_ssrc, kOutgoingPacket);
extension_maps[rtx_stream] =
- RtpHeaderExtensionMap(configs[i].rtp_extensions);
+ RtpHeaderExtensionMap(config.rtp_extensions);
video_ssrcs_.insert(rtx_stream);
rtx_ssrcs_.insert(rtx_stream);
}
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698