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

Fixes minor issue in CreateDeviceInternal (ADM2 on Windows)

Browse Source 
Resolves an old (unresolved) review comment in
https://webrtc-review.googlesource.com/c/src/+/160050/6/modules/audio_device/win/core_audio_utility_win.cc#325

The mistake was pointed out by dkirovbroadsoft@gmail.com. Thanks!

Tbr: thaloun@chromium.org
Bug: webrtc:11107
Change-Id: Ib732eaea8b07c6d6fb0b8963b00c3b009ccb8fee
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/217120
Reviewed-by: Henrik Andreassson <henrika@webrtc.org>
Commit-Queue: Henrik Andreassson <henrika@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#33893}
This commit is contained in:
henrika
2021-05-03 09:16:44 +02:00
committed by WebRTC LUCI CQ
parent e75b7f2eb2
commit 410d70d965
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
modules/audio_device/win/core_audio_utility_win.cc
View File

@ -323,7 +323,7 @@ ComPtr<IMMDevice> CreateDeviceInternal(const std::string& device_id,
// Verify that the audio endpoint device is active, i.e., that the audio
// adapter that connects to the endpoint device is present and enabled.
if (SUCCEEDED(error.Error()) && !audio_endpoint_device.Get() &&
if (SUCCEEDED(error.Error()) && audio_endpoint_device.Get() &&
!IsDeviceActive(audio_endpoint_device.Get())) {
RTC_LOG(LS_WARNING) << "Selected endpoint device is not active";
audio_endpoint_device.Reset();