zqz/platform-external-webrtc
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix more -Wextra-semi.

Browse Source 
Chromium has enabled -Wextra-semi on Android, iOS and Windwos builds:
https://chromium-review.googlesource.com/c/1489138
https://chromium-review.googlesource.com/c/1489180
https://chromium-review.googlesource.com/c/1489102

This CL fixes some minor problems in WebRTC and it is a follow-up of
https://webrtc-review.googlesource.com/c/src/+/123560 and
https://webrtc-review.googlesource.com/c/124440.

Bug: webrtc:10355, chromium:926235
Change-Id: Ie9e49077a72c783c3e0fbc21bbe237d7338407e4
Reviewed-on: https://webrtc-review.googlesource.com/c/124680
Reviewed-by: Karl Wiberg <kwiberg@webrtc.org>
Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#26873}
This commit is contained in:
Mirko Bonadei
2019-02-27 12:01:51 +01:00
committed by Commit Bot
parent 5cceaa1294
commit 28221dee85
3 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
modules/video_capture/windows/sink_filter_ds.h
View File

@ -67,7 +67,7 @@ class CaptureSinkFilter : public CBaseFilter {
// -------------------------------------------------------------------- // --------------------------------------------------------------------
// COM interfaces // COM interfaces
DECLARE_IUNKNOWN; DECLARE_IUNKNOWN
STDMETHODIMP SetMatchingMediaType(const VideoCaptureCapability& capability); STDMETHODIMP SetMatchingMediaType(const VideoCaptureCapability& capability);
// -------------------------------------------------------------------- // --------------------------------------------------------------------

2
sdk/android/src/jni/pc/rtp_transceiver.cc
View File

@ -65,7 +65,7 @@ ScopedJavaLocalRef<jobject> NativeToJavaRtpTransceiver(
JavaRtpTransceiverGlobalOwner::JavaRtpTransceiverGlobalOwner( JavaRtpTransceiverGlobalOwner::JavaRtpTransceiverGlobalOwner(
JNIEnv* env, JNIEnv* env,
const JavaRef<jobject>& j_transceiver) const JavaRef<jobject>& j_transceiver)
: j_transceiver_(env, j_transceiver){}; : j_transceiver_(env, j_transceiver) {}
JavaRtpTransceiverGlobalOwner::JavaRtpTransceiverGlobalOwner( JavaRtpTransceiverGlobalOwner::JavaRtpTransceiverGlobalOwner(
JavaRtpTransceiverGlobalOwner&& other) = default; JavaRtpTransceiverGlobalOwner&& other) = default;

2
sdk/objc/native/src/objc_frame_buffer.mm
View File

@ -22,7 +22,7 @@ class ObjCI420FrameBuffer : public I420BufferInterface {
public: public:
explicit ObjCI420FrameBuffer(id<RTCI420Buffer> frame_buffer) explicit ObjCI420FrameBuffer(id<RTCI420Buffer> frame_buffer)
: frame_buffer_(frame_buffer), width_(frame_buffer.width), height_(frame_buffer.height) {} : frame_buffer_(frame_buffer), width_(frame_buffer.width), height_(frame_buffer.height) {}
~ObjCI420FrameBuffer() override{}; ~ObjCI420FrameBuffer() override {}
int width() const override { return width_; } int width() const override { return width_; }