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.

1 files changed, 1 insertions, 1 deletions

diff --git a/audio/out/ao_openal.c b/audio/out/ao_openal.c
index 715ffddceb..6ae91162d3 100644
--- a/audio/out/ao_openal.c
+++ b/audio/out/ao_openal.c
@@ -197,7 +197,7 @@ static int init(struct ao *ao)
         ao->samplerate = freq;
 
     p->al_format = AL_FALSE;
-    int try_formats[AF_FORMAT_COUNT];
+    int try_formats[AF_FORMAT_COUNT + 1];
     af_get_best_sample_formats(ao->format, try_formats);
     for (int n = 0; try_formats[n]; n++) {
         p->al_format = get_al_format(try_formats[n]);