We are currently migrating Bugzilla to GitHub issues.
Any changes made to the bug tracker now will be lost, so please do not post new bugs or make changes to them.
When we're done, all bug URLs will redirect to their equivalent location on the new bug tracker.

[patch] Patch

sdl-alsa.diff (text/plain), 3.60 KB, created by Tilman Sauerbeck on 2009-05-24 04:46:55 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Tilman Sauerbeck

Created: 2009-05-24 04:46:55 UTC

Size: 3.60 KB

patch

obsolete

>diff -aur SDL-1.2.13.orig/src/audio/alsa/SDL_alsa_audio.c SDL-1.2.13/src/audio/alsa/SDL_alsa_audio.c
>--- SDL-1.2.13.orig/src/audio/alsa/SDL_alsa_audio.c	2007-12-31 05:47:57.000000000 +0100
>+++ SDL-1.2.13/src/audio/alsa/SDL_alsa_audio.c	2009-05-24 13:35:23.007636110 +0200
>@@ -75,10 +75,10 @@
> static int (*SDL_NAME(snd_pcm_hw_params_set_format))(snd_pcm_t *pcm, snd_pcm_hw_params_t *params, snd_pcm_format_t val);
> static int (*SDL_NAME(snd_pcm_hw_params_set_channels))(snd_pcm_t *pcm, snd_pcm_hw_params_t *params, unsigned int val);
> static int (*SDL_NAME(snd_pcm_hw_params_get_channels))(const snd_pcm_hw_params_t *params);
>-static unsigned int (*SDL_NAME(snd_pcm_hw_params_set_rate_near))(snd_pcm_t *pcm, snd_pcm_hw_params_t *params, unsigned int val, int *dir);
>-static snd_pcm_uframes_t (*SDL_NAME(snd_pcm_hw_params_set_period_size_near))(snd_pcm_t *pcm, snd_pcm_hw_params_t *params, snd_pcm_uframes_t val, int *dir);
>+static int (*SDL_NAME(snd_pcm_hw_params_set_rate_near))(snd_pcm_t *pcm, snd_pcm_hw_params_t *params, unsigned int *val, int *dir);
>+static int (*SDL_NAME(snd_pcm_hw_params_set_period_size_near))(snd_pcm_t *pcm, snd_pcm_hw_params_t *params, snd_pcm_uframes_t *val, int *dir);
> static snd_pcm_sframes_t (*SDL_NAME(snd_pcm_hw_params_get_period_size))(const snd_pcm_hw_params_t *params);
>-static unsigned int (*SDL_NAME(snd_pcm_hw_params_set_periods_near))(snd_pcm_t *pcm, snd_pcm_hw_params_t *params, unsigned int val, int *dir);
>+static int (*SDL_NAME(snd_pcm_hw_params_set_periods_near))(snd_pcm_t *pcm, snd_pcm_hw_params_t *params, unsigned int *val, int *dir);
> static int (*SDL_NAME(snd_pcm_hw_params_get_periods))(snd_pcm_hw_params_t *params);
> static int (*SDL_NAME(snd_pcm_hw_params))(snd_pcm_t *pcm, snd_pcm_hw_params_t *params);
> /*
>@@ -374,6 +374,8 @@
> 	snd_pcm_sw_params_t *swparams;
> 	snd_pcm_format_t     format;
> 	snd_pcm_uframes_t    frames;
>+	unsigned int         rate;
>+	unsigned int         periods;
> 	Uint16               test_format;
> 
> 	/* Open the audio device */
>@@ -456,19 +458,29 @@
> 	}
> 
> 	/* Set the audio rate */
>-	status = SDL_NAME(snd_pcm_hw_params_set_rate_near)(pcm_handle, hwparams, spec->freq, NULL);
>+	rate = spec->freq;
>+
>+	status = SDL_NAME(snd_pcm_hw_params_set_rate_near)(pcm_handle, hwparams, &rate, NULL);
> 	if ( status < 0 ) {
> 		SDL_SetError("Couldn't set audio frequency: %s", SDL_NAME(snd_strerror)(status));
> 		ALSA_CloseAudio(this);
> 		return(-1);
> 	}
>-	spec->freq = status;
>+	spec->freq = rate;
> 
> 	/* Set the buffer size, in samples */
> 	frames = spec->samples;
>-	frames = SDL_NAME(snd_pcm_hw_params_set_period_size_near)(pcm_handle, hwparams, frames, NULL);
>+	status = SDL_NAME(snd_pcm_hw_params_set_period_size_near)(pcm_handle, hwparams, &frames, NULL);
>+	if ( status < 0 ) {
>+		SDL_SetError("Couldn't set audio frequency: %s", SDL_NAME(snd_strerror)(status));
>+		ALSA_CloseAudio(this);
>+		return(-1);
>+	}
>+
> 	spec->samples = frames;
>-	SDL_NAME(snd_pcm_hw_params_set_periods_near)(pcm_handle, hwparams, 2, NULL);
>+
>+	periods = 2;
>+	SDL_NAME(snd_pcm_hw_params_set_periods_near)(pcm_handle, hwparams, &periods, NULL);
> 
> 	/* "set" the hardware with the desired parameters */
> 	status = SDL_NAME(snd_pcm_hw_params)(pcm_handle, hwparams);
>diff -aur SDL-1.2.13.orig/src/audio/alsa/SDL_alsa_audio.h SDL-1.2.13/src/audio/alsa/SDL_alsa_audio.h
>--- SDL-1.2.13.orig/src/audio/alsa/SDL_alsa_audio.h	2007-12-31 05:47:57.000000000 +0100
>+++ SDL-1.2.13/src/audio/alsa/SDL_alsa_audio.h	2009-05-24 13:14:50.751632465 +0200
>@@ -24,8 +24,6 @@
> #ifndef _ALSA_PCM_audio_h
> #define _ALSA_PCM_audio_h
> 
>-#define ALSA_PCM_OLD_HW_PARAMS_API
>-#define ALSA_PCM_OLD_SW_PARAMS_API
> #include <alsa/asoundlib.h>
> 
> #include "../SDL_sysaudio.h"

Actions: View | Diff

Attachments on bug 745: 327 | 340 | 341

We are currently migrating Bugzilla to GitHub issues.Any changes made to the bug tracker now will be lost, so please do not post new bugs or make changes to them.When we're done, all bug URLs will redirect to their equivalent location on the new bug tracker.

We are currently migrating Bugzilla to GitHub issues.
Any changes made to the bug tracker now will be lost, so please do not post new bugs or make changes to them.
When we're done, all bug URLs will redirect to their equivalent location on the new bug tracker.