From 67f2538c41664f66f06d42df8e875e99883ea962 Mon Sep 17 00:00:00 2001 From: "Ryan C. Gordon" Date: Mon, 1 Aug 2016 13:32:27 -0400 Subject: [PATCH] audio: changed some internal ints to be SDL_bools. --- src/audio/SDL_audio.c | 17 +++++++++-------- src/audio/SDL_sysaudio.h | 9 ++++----- src/audio/qsa/SDL_qsa_audio.c | 2 +- src/audio/qsa/SDL_qsa_audio.h | 2 +- 4 files changed, 15 insertions(+), 15 deletions(-) diff --git a/src/audio/SDL_audio.c b/src/audio/SDL_audio.c index b7dab5c947642..8fee9dccc18b6 100644 --- a/src/audio/SDL_audio.c +++ b/src/audio/SDL_audio.c @@ -373,7 +373,7 @@ void SDL_OpenedAudioDeviceDisconnected(SDL_AudioDevice *device) /* Ends the audio callback and mark the device as STOPPED, but the app still needs to close the device to free resources. */ current_audio.impl.LockDevice(device); - device->enabled = 0; + device->enabled = SDL_FALSE; current_audio.impl.UnlockDevice(device); /* Post the event, if desired */ @@ -873,7 +873,7 @@ SDL_GetAudioDeviceName(int index, int iscapture) static void close_audio_device(SDL_AudioDevice * device) { - device->enabled = 0; + device->enabled = SDL_FALSE; SDL_AtomicSet(&device->shutdown, 1); if (device->thread != NULL) { SDL_WaitThread(device->thread, NULL); @@ -887,7 +887,7 @@ close_audio_device(SDL_AudioDevice * device) } if (device->opened) { current_audio.impl.CloseDevice(device); - device->opened = 0; + device->opened = SDL_FALSE; } free_audio_queue(device->buffer_queue_head); @@ -1074,11 +1074,12 @@ open_audio_device(const char *devname, int iscapture, return 0; } SDL_zerop(device); + SDL_AtomicSet(&device->shutdown, 0); /* just in case. */ device->id = id + 1; device->spec = *obtained; - device->enabled = 1; - device->paused = 1; - device->iscapture = iscapture; + device->enabled = SDL_TRUE; + device->paused = SDL_TRUE; + device->iscapture = iscapture ? SDL_TRUE : SDL_FALSE; /* Create a mutex for locking the sound buffers */ if (!current_audio.impl.SkipMixerLock) { @@ -1094,7 +1095,7 @@ open_audio_device(const char *devname, int iscapture, close_audio_device(device); return 0; } - device->opened = 1; + device->opened = SDL_TRUE; /* See if we need to do any conversion */ build_cvt = SDL_FALSE; @@ -1278,7 +1279,7 @@ SDL_PauseAudioDevice(SDL_AudioDeviceID devid, int pause_on) SDL_AudioDevice *device = get_audio_device(devid); if (device) { current_audio.impl.LockDevice(device); - device->paused = pause_on; + device->paused = pause_on ? SDL_TRUE : SDL_FALSE; current_audio.impl.UnlockDevice(device); } } diff --git a/src/audio/SDL_sysaudio.h b/src/audio/SDL_sysaudio.h index 3fb4c5cb4987b..fcf441d0ef595 100644 --- a/src/audio/SDL_sysaudio.h +++ b/src/audio/SDL_sysaudio.h @@ -158,11 +158,10 @@ struct SDL_AudioDevice /* Current state flags */ SDL_atomic_t shutdown; /* true if we are signaling the play thread to end. */ - /* !!! FIXME: these should be SDL_bool */ - int iscapture; - int enabled; /* true if device is functioning and connected. */ - int paused; - int opened; + SDL_bool iscapture; + SDL_bool enabled; /* true if device is functioning and connected. */ + SDL_bool paused; + SDL_bool opened; /* Fake audio buffer for when the audio hardware is busy */ Uint8 *fake_stream; diff --git a/src/audio/qsa/SDL_qsa_audio.c b/src/audio/qsa/SDL_qsa_audio.c index 1899ca6d94b1c..1742fe3f619ea 100644 --- a/src/audio/qsa/SDL_qsa_audio.c +++ b/src/audio/qsa/SDL_qsa_audio.c @@ -371,7 +371,7 @@ QSA_OpenDevice(_THIS, void *handle, const char *devname, int iscapture) QSA_InitAudioParams(&cparams); /* Initialize channel direction: capture or playback */ - this->hidden->iscapture = iscapture; + this->hidden->iscapture = iscapture ? SDL_TRUE : SDL_FALSE; if (device != NULL) { /* Open requested audio device */ diff --git a/src/audio/qsa/SDL_qsa_audio.h b/src/audio/qsa/SDL_qsa_audio.h index 53d37e96fd41f..19a2215aada77 100644 --- a/src/audio/qsa/SDL_qsa_audio.h +++ b/src/audio/qsa/SDL_qsa_audio.h @@ -34,7 +34,7 @@ struct SDL_PrivateAudioData { /* SDL capture state */ - int iscapture; + SDL_bool iscapture; /* The audio device handle */ int cardno;