From 057ce1de459775affd165fd9bcb531dc44d09a3d Mon Sep 17 00:00:00 2001 From: Oskar Sundbom Date: Thu, 16 Nov 2017 10:56:15 +0100 Subject: [PATCH] Optional: Use nullopt and implicit construction in /sdk/android/src/jni/androidvideotracksource.h MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 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=sakal@webrtc.org Bug: None Change-Id: Ib01eb9ef7b3b76c0f55890d7ea1b8dfcd30f0617 Reviewed-on: https://webrtc-review.googlesource.com/23607 Reviewed-by: Sami Kalliomäki Commit-Queue: Oskar Sundbom Cr-Commit-Position: refs/heads/master@{#20845} --- sdk/android/src/jni/androidvideotracksource.h | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/sdk/android/src/jni/androidvideotracksource.h b/sdk/android/src/jni/androidvideotracksource.h index 8836c11176..524c36d837 100644 --- a/sdk/android/src/jni/androidvideotracksource.h +++ b/sdk/android/src/jni/androidvideotracksource.h @@ -38,9 +38,7 @@ class AndroidVideoTrackSource : 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; } // Called by the native capture observer void SetState(SourceState state);