From d56efd147a923a2d91bfe94b88e973043056f8c2 Mon Sep 17 00:00:00 2001 From: Aaron Vaage Date: Wed, 23 May 2018 14:59:41 -0700 Subject: [PATCH] Disallow Copy and Assign For WebVTT Output We don't want to allow any handler to be copyable or assigned-over so this change enforces that for the webvtt output handler. Change-Id: Ie0d59d6dbfb7a5e00bb4dd1422cd696d1a2d6072 --- packager/media/formats/webvtt/webvtt_output_handler.h | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/packager/media/formats/webvtt/webvtt_output_handler.h b/packager/media/formats/webvtt/webvtt_output_handler.h index 5518504649..9ee0140d4d 100644 --- a/packager/media/formats/webvtt/webvtt_output_handler.h +++ b/packager/media/formats/webvtt/webvtt_output_handler.h @@ -23,10 +23,11 @@ class WebVttTextOutputHandler : public MediaHandler { public: WebVttTextOutputHandler(const MuxerOptions& muxer_options, std::unique_ptr muxer_listener); - // TODO(vaage): Add missing default destructor. + virtual ~WebVttTextOutputHandler() = default; private: - // TODO(vaage): Add missing deleted copy and assign constructors. + WebVttTextOutputHandler(const WebVttTextOutputHandler&) = delete; + WebVttTextOutputHandler& operator=(const WebVttTextOutputHandler&) = delete; Status InitializeInternal() override; Status Process(std::unique_ptr stream_data) override;