From 19b3a554e8dc9e484906060576a5151e794c34cf Mon Sep 17 00:00:00 2001 From: deadbeef Date: Fri, 16 Jun 2017 20:19:08 -0700 Subject: [PATCH] Fixing incorrect use of erase/remove idiom. In this case it wasn't an issue, because only one result would be found by remove_if, but might as well fix it just in case. BUG=None TBR=pthatcher@webrtc.org Review-Url: https://codereview.webrtc.org/2945723002 Cr-Commit-Position: refs/heads/master@{#18641} --- webrtc/pc/statscollector.cc | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/webrtc/pc/statscollector.cc b/webrtc/pc/statscollector.cc index c0b18de6a1..f4db313b44 100644 --- a/webrtc/pc/statscollector.cc +++ b/webrtc/pc/statscollector.cc @@ -435,11 +435,13 @@ void StatsCollector::AddLocalAudioTrack(AudioTrackInterface* audio_track, void StatsCollector::RemoveLocalAudioTrack(AudioTrackInterface* audio_track, uint32_t ssrc) { RTC_DCHECK(audio_track != NULL); - local_audio_tracks_.erase(std::remove_if(local_audio_tracks_.begin(), - local_audio_tracks_.end(), - [audio_track, ssrc](const LocalAudioTrackVector::value_type& track) { - return track.first == audio_track && track.second == ssrc; - })); + local_audio_tracks_.erase( + std::remove_if( + local_audio_tracks_.begin(), local_audio_tracks_.end(), + [audio_track, ssrc](const LocalAudioTrackVector::value_type& track) { + return track.first == audio_track && track.second == ssrc; + }), + local_audio_tracks_.end()); } void StatsCollector::GetStats(MediaStreamTrackInterface* track,