From d21bf131bd3a988f7b21d9e8830819b804729b42 Mon Sep 17 00:00:00 2001 From: Oskar Sundbom Date: Thu, 16 Nov 2017 10:54:04 +0100 Subject: [PATCH] Optional: Use nullopt and implicit construction in /sdk/objc Changes places where we explicitly construct an Optional to instead use nullopt or the requisite value type only. This CL was uploaded by git cl split. R=magjed@webrtc.org Bug: None Change-Id: I78842b6bb8ae345bcb852feee3908fdaf955c664 Reviewed-on: https://webrtc-review.googlesource.com/23574 Reviewed-by: Magnus Jedvert Commit-Queue: Oskar Sundbom Cr-Commit-Position: refs/heads/master@{#20956} --- sdk/objc/Framework/Classes/Video/objcvideotracksource.h | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/sdk/objc/Framework/Classes/Video/objcvideotracksource.h b/sdk/objc/Framework/Classes/Video/objcvideotracksource.h index 27c7295dda..656fd66118 100644 --- a/sdk/objc/Framework/Classes/Video/objcvideotracksource.h +++ b/sdk/objc/Framework/Classes/Video/objcvideotracksource.h @@ -30,9 +30,7 @@ class ObjcVideoTrackSource : public rtc::AdaptedVideoTrackSource { // Indicates that the encoder should denoise video before encoding it. // If it is not set, the default configuration is used which is different // depending on video codec. - rtc::Optional needs_denoising() const override { - return rtc::Optional(false); - } + rtc::Optional needs_denoising() const override { return false; } SourceState state() const override { return SourceState::kLive; }