[MPlayer-dev-eng] [PATCH] add OpenAL backend device support
Diego Biurrun
diego at biurrun.de
Mon Sep 13 17:02:23 CEST 2010
On Mon, Sep 13, 2010 at 12:08:47PM +0200, Erik Hofman wrote:
>
> For my own purpose I needed to specify a different OpenAL device instead
> of the default one. This patch add support for the device="" parameter
> for the openal backend as well as adding support for the help
> parameter, showing a list of available devices.
>
> --- libao2/ao_openal.c (revision 32219)
> +++ libao2/ao_openal.c (working copy)
> @@ -62,7 +62,9 @@
> static int unqueue_buf[MAX_CHANS];
> static int16_t *tmpbuf;
>
> +static void printList(const char *header, char separator, const char eol, const char *extensions);
Avoid this ugly forward declaration by placing the functions in
the appropriate order.
> @@ -252,3 +280,71 @@
> +
> +static void
> +printChar (int c, int *width)
> +{
Please respect the coding style of the rest of the file and format
your function headers accordingly.
> + while (1)
> + {
> + if (extensions[end] == separator || extensions[end] == '\0')
> + {
> + if (width + end - start + 2 > maxmimumWidth)
> + {
> + printChar ('\n', &width);
> + indent (&width);
> + }
> + while (start < end)
> + {
> + printChar (extensions[start], &width);
> + start++;
> + }
> + if (extensions[end] == '\0')
> + {
same here; inconsistent { placement
Diego
More information about the MPlayer-dev-eng
mailing list