Support 4 channel mic in Windows Core Audio

webrtc/audio/utility/audio_frame_operations.h

 /*
  *  Copyright (c) 2012 The WebRTC project authors. All Rights Reserved.
  *
  *  Use of this source code is governed by a BSD-style license
  *  that can be found in the LICENSE file in the root of the source
  *  tree. An additional intellectual property rights grant can be found
  *  in the file PATENTS.  All contributing project authors may
  *  be found in the AUTHORS file in the root of the source tree.
  */
 
@@ skipping 36 matching lines @@
   // Downmixes stereo |src_audio| to mono |dst_audio|. This is an in-place
   // operation, meaning |src_audio| and |dst_audio| may point to the same
   // buffer.
   static void StereoToMono(const int16_t* src_audio,
                            size_t samples_per_channel,
                            int16_t* dst_audio);
   // |frame.num_channels_| will be updated. This version checks that
   // |num_channels_| is stereo.
   static int StereoToMono(AudioFrame* frame);
 
+  // Downmixes 4 channels |src_audio| to stereo |dst_audio|. This is an in-place
+  // operation, meaning |src_audio| and |dst_audio| may point to the same
+  // buffer.
+  static void QuadToStereo(const int16_t* src_audio,
+                           size_t samples_per_channel,
+                           int16_t* dst_audio);

[aleloi, 2017/02/24 10:39:31]

Add blank line

[jens.nielsen, 2017/02/24 14:55:49]

Done.

+  // |frame.num_channels_| will be updated. This version checks that
+  // |num_channels_| is 4 channels.
+  static int QuadToStereo(AudioFrame* frame);
+
+  // Downmixes 4 channels |src_audio| to mono |dst_audio|. This is an in-place
+  // operation, meaning |src_audio| and |dst_audio| may point to the same
+  // buffer.
+  static void QuadToMono(const int16_t* src_audio,
+                         size_t samples_per_channel,
+                         int16_t* dst_audio);

[aleloi, 2017/02/24 10:39:31]

Add blank line

[jens.nielsen, 2017/02/24 14:55:48]

Done.

+  // |frame.num_channels_| will be updated. This version checks that
+  // |num_channels_| is stereo.

[aleloi, 2017/02/24 10:39:31]

Shouldn't |num_channels_| be 4 here and not stereo? Or am I missing something?

[jens.nielsen, 2017/02/24 14:55:48]

Done. (Silly copy-paste error, sorry about that)

+  static int QuadToMono(AudioFrame* frame);
+
+  // Downmixes |src_channels| |src_audio| to |dst_channels| |dst_audio|.
+  // This is an in-place operation, meaning |src_audio| and |dst_audio|
+  // may point to the same buffer.

[hlundin-webrtc, 2017/02/24 09:46:36]

Comment on which combinations of src_channels and dst_channels are supported.

[jens.nielsen, 2017/02/24 14:55:49]

Done.

+  static void AudioFrameOperations::DownmixChannels(const int16_t* src_audio,
+                                                    size_t src_channels,
+                                                    size_t samples_per_channel,
+                                                    int16_t* dst_audio,
+                                                    size_t dst_channels);

[aleloi, 2017/02/24 10:39:31]

Please move the dst_audio to the end.

Quote from style guide: "When defining a function, parameter order is: inputs,
then outputs."

https://google.github.io/styleguide/cppguide.html#Function_Parameter_Ordering

[jens.nielsen, 2017/02/24 14:55:49]

Done.

+
+  // |frame.num_channels_| will be updated. This version checks that
+  // |num_channels_| and |dst_channels| are valid and performs relevant
+  // downmix.

[hlundin-webrtc, 2017/02/24 09:46:36]

Comment on which combinations of src_channels and dst_channels are supported.

[jens.nielsen, 2017/02/24 14:55:49]

Done.

+  static int AudioFrameOperations::DownmixChannels(AudioFrame* frame,
+                                                   size_t dst_channels);

[aleloi, 2017/02/24 10:39:31]

Please swap function parameters here as well. |dst_channels| is input, |frame|
is both, so it should be last.

[jens.nielsen, 2017/02/24 14:55:49]

Done.

+
   // Swap the left and right channels of |frame|. Fails silently if |frame| is
   // not stereo.
   static void SwapStereoChannels(AudioFrame* frame);
 
   // Conditionally zero out contents of |frame| for implementing audio mute:
   //  |previous_frame_muted| &&  |current_frame_muted| - Zero out whole frame.
   //  |previous_frame_muted| && !|current_frame_muted| - Fade-in at frame start.
   // !|previous_frame_muted| &&  |current_frame_muted| - Fade-out at frame end.
   // !|previous_frame_muted| && !|current_frame_muted| - Leave frame untouched.
   static void Mute(AudioFrame* frame,
                    bool previous_frame_muted,
                    bool current_frame_muted);
 
   // Zero out contents of frame.
   static void Mute(AudioFrame* frame);
 
   // Halve samples in |frame|.
   static void ApplyHalfGain(AudioFrame* frame);
 
   static int Scale(float left, float right, AudioFrame& frame);
 
   static int ScaleWithSat(float scale, AudioFrame& frame);
 };
 
 }  // namespace webrtc
 
 #endif  // WEBRTC_AUDIO_UTILITY_AUDIO_FRAME_OPERATIONS_H_