Devices

Timestamp:

Jun 26, 2017 12:41:12 PM (8 years ago)

Author:

vboxsync

svn:sync-xref-src-repo-rev:

116367

Message:

Audio/DrvAudioCommon.cpp: Relax audio channels check in DrvAudioHlpPCMPropsAreValid().

File:

trunk/src/VBox/Devices/Audio/DrvAudioCommon.cpp

-              r65738
+              r67606
  * Returns @c true if properties are valid, @c false if not.
  * @param   pProps              PCM properties to check.
+ *
- * @remarks Does *not* support surround (> 2 channels) yet! This is intentional, as
- *          we consider surround support as experimental / not enabled by default for now.
  */
 bool DrvAudioHlpPCMPropsAreValid(const PPDMAUDIOPCMPROPS pProps)
 …
     AssertPtrReturn(pProps, false);
+    bool fValid = (   pProps->cChannels == 1
+                   || pProps->cChannels == 2); /* Either stereo (2) or mono (1), per stream. */
+    /* Minimum 1 channel (mono), maximum 7.1 (= 8) channels. */
+    bool fValid = (   pProps->cChannels >= 1
+                   && pProps->cChannels <= 8);
     if (fValid)
 …
  * Returns @c true if configuration is valid, @c false if not.
  * @param   pCfg                Stream configuration to check.
+ *
- * @remarks Does *not* support surround (> 2 channels) yet! This is intentional, as
- *          we consider surround support as experimental / not enabled by default for now.
  */
 bool DrvAudioHlpStreamCfgIsValid(const PPDMAUDIOSTREAMCFG pCfg)

Note: See TracChangeset for help on using the changeset viewer.