commit 2a5ce2bcf8bf87545dc98084c27b622a6e089f2f
author Danil Chapovalov <danilchap@webrtc.org> Wed Feb 07 09:38:31 2018 +0100
committer Commit Bot <commit-bot@chromium.org> Wed Feb 07 09:48:28 2018 +0000
tree d3a87bc6bed9c8723166bb3eb36bcd7584076f29
parent 740f8e72dff1664bae5e734c25976cdb27169cdc

Fix clang style errors in rtp_rtcp and dependant targets

Mark functions with override instead of virtual.
Add explicit non-trivial constructors/assign operators/destructors.
Define them in .cc files instead of inlining
use auto* instead of auto when deduced type is raw pointer

Bug: webrtc:163
Change-Id: I4d8a05d6a64fcc2ca16d02c5fcf9488fda832a6d
Reviewed-on: https://webrtc-review.googlesource.com/48781
Commit-Queue: Danil Chapovalov <danilchap@webrtc.org>
Reviewed-by: Karl Wiberg <kwiberg@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#21927}

modules/rtp_rtcp/source/rtp_header_parser.cc

diff --git a/modules/rtp_rtcp/source/rtp_header_parser.cc b/modules/rtp_rtcp/source/rtp_header_parser.cc
index 5d2971a..df68f74 100644
--- a/modules/rtp_rtcp/source/rtp_header_parser.cc
+++ b/modules/rtp_rtcp/source/rtp_header_parser.cc
@@ -18,7 +18,7 @@
 class RtpHeaderParserImpl : public RtpHeaderParser {
  public:
   RtpHeaderParserImpl();
-  virtual ~RtpHeaderParserImpl() {}
+  ~RtpHeaderParserImpl() override = default;
 
   bool Parse(const uint8_t* packet,
              size_t length,