summaryrefslogtreecommitdiffstats
path: root/audio/out/ao_coreaudio.c
diff options
context:
space:
mode:
authorwm4 <wm4@nowhere>2015-06-26 15:49:40 +0200
committerwm4 <wm4@nowhere>2015-06-26 15:56:19 +0200
commit8134a0601b0cafbd466bf2b44f15a8592ce7b05f (patch)
treea82f45af6e6e47f34846c8cf25ccc4e34da7e27f /audio/out/ao_coreaudio.c
parent3c61e6eb4e2d340e959d5395ec7c61902fa28b1d (diff)
downloadmpv-8134a0601b0cafbd466bf2b44f15a8592ce7b05f.tar.bz2
mpv-8134a0601b0cafbd466bf2b44f15a8592ce7b05f.tar.xz
ao_coreaudio: explicitly skip input streams
This may or may not fix some issues with the format switching code. Actually, it seems somewhat unlikely, but then checking the stream type isn't incorrect either, and is probably something the API user should always be doing.
Diffstat (limited to 'audio/out/ao_coreaudio.c')
-rw-r--r--audio/out/ao_coreaudio.c18
1 files changed, 14 insertions, 4 deletions
diff --git a/audio/out/ao_coreaudio.c b/audio/out/ao_coreaudio.c
index ea780a9d1e..d3d2bf046e 100644
--- a/audio/out/ao_coreaudio.c
+++ b/audio/out/ao_coreaudio.c
@@ -192,18 +192,28 @@ static void init_physical_format(struct ao *ao)
&streams, &n_streams);
CHECK_CA_ERROR("could not get number of streams");
+ MP_VERBOSE(ao, "Found %zd substream(s).\n", n_streams);
+
for (int i = 0; i < n_streams; i++) {
AudioStreamRangedDescription *formats;
size_t n_formats;
- err = CA_GET_ARY(streams[i],
- kAudioStreamPropertyAvailablePhysicalFormats,
- &formats, &n_formats);
+ MP_VERBOSE(ao, "Looking at formats in substream %d...\n", i);
+
+ err = CA_GET_ARY(streams[i], kAudioStreamPropertyAvailablePhysicalFormats,
+ &formats, &n_formats);
if (!CHECK_CA_WARN("could not get number of stream formats"))
continue; // try next one
- MP_VERBOSE(ao, "Looking at formats in substream %d...\n", i);
+
+ uint32_t direction;
+ err = CA_GET(streams[i], kAudioStreamPropertyDirection, &direction);
+ CHECK_CA_ERROR("could not get stream direction");
+ if (direction != 0) {
+ MP_VERBOSE(ao, "Not an output stream.\n");
+ continue;
+ }
AudioStreamBasicDescription best_asbd = {0};