From da825b1b1230202435601c35930afc99cf2eef6a Mon Sep 17 00:00:00 2001 From: "Piotr (Peter) Slatala" Date: Wed, 13 Feb 2019 14:38:44 -0800 Subject: [PATCH] Replace NOTREACHED with a break. MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The old format is still used. Bug: None Change-Id: I2c8c9a8c7b09282ba8c375403cbee7fd7ab2d056 Reviewed-on: https://webrtc-review.googlesource.com/c/122960 Reviewed-by: Björn Terelius Commit-Queue: Björn Terelius Cr-Commit-Position: refs/heads/master@{#26679} --- logging/rtc_event_log/encoder/rtc_event_log_encoder_legacy.cc | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/logging/rtc_event_log/encoder/rtc_event_log_encoder_legacy.cc b/logging/rtc_event_log/encoder/rtc_event_log_encoder_legacy.cc index 6aab870ce9..4e8529cd54 100644 --- a/logging/rtc_event_log/encoder/rtc_event_log_encoder_legacy.cc +++ b/logging/rtc_event_log/encoder/rtc_event_log_encoder_legacy.cc @@ -365,7 +365,8 @@ std::string RtcEventLogEncoderLegacy::Encode(const RtcEvent& event) { case RtcEvent::Type::GenericPacketReceived: case RtcEvent::Type::GenericPacketSent: case RtcEvent::Type::GenericAckReceived: - RTC_NOTREACHED(); + // These are unsupported in the old format. + break; } int event_type = static_cast(event.GetType());