diff options
author | wm4 <wm4@nowhere> | 2018-01-24 00:02:13 +0100 |
---|---|---|
committer | Kevin Mitchell <kevmitch@gmail.com> | 2018-01-25 20:18:32 -0800 |
commit | d36ff64b29031f402184e4946969810e7d9db0e0 (patch) | |
tree | e57f501ce20b384d2183af98f3220ddaa4e7d602 /audio/out/ao_wasapi_utils.c | |
parent | 3dd5621cecdd41c86117d70f93a24aede5e50bbc (diff) | |
download | mpv-d36ff64b29031f402184e4946969810e7d9db0e0.tar.bz2 mpv-d36ff64b29031f402184e4946969810e7d9db0e0.tar.xz |
audio: fix annyoing af_get_best_sample_formats() definition
The af_get_best_sample_formats() function had an argument of
int[AF_FORMAT_COUNT], which is slightly incorrect, because it's 0
terminated and should in theory have AF_FORMAT_COUNT+1 entries. It won't
actually write this many formats (since some formats are fundamentally
incompatible), but it still feels annoying and incorrect. So fix it, and
require that callers pass an AF_FORMAT_COUNT+1 array.
Note that the array size has no meaning in C function arguments (just
another issue with C static arrays being weird and stupid), so get rid
of it completely.
Not changing the af_lavcac3enc use, since that is rewritten in another
branch anyway.
Diffstat (limited to 'audio/out/ao_wasapi_utils.c')
-rw-r--r-- | audio/out/ao_wasapi_utils.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/audio/out/ao_wasapi_utils.c b/audio/out/ao_wasapi_utils.c index 471cc847ac..55a16da812 100644 --- a/audio/out/ao_wasapi_utils.c +++ b/audio/out/ao_wasapi_utils.c @@ -97,7 +97,7 @@ static const struct wasapi_sample_fmt wasapi_formats[] = { static void wasapi_get_best_sample_formats( int src_format, struct wasapi_sample_fmt *out_formats) { - int mp_formats[AF_FORMAT_COUNT]; + int mp_formats[AF_FORMAT_COUNT + 1]; af_get_best_sample_formats(src_format, mp_formats); for (int n = 0; mp_formats[n]; n++) { for (int i = 0; wasapi_formats[i].mp_format; i++) { |