aboutsummaryrefslogtreecommitdiff
path: root/pcmalsa.cpp
diff options
context:
space:
mode:
authorJavier <dev.git@javispedro.com>2022-01-30 20:02:28 +0100
committerJavier <dev.git@javispedro.com>2022-01-30 20:02:28 +0100
commit7bb77279bf5b76902c167923d57800e1f6f6073e (patch)
tree9c23a39b3ea2e6af094240780ce21206602a0c11 /pcmalsa.cpp
parent54b754ce040d5549d5c58428d2b9c095601e98dc (diff)
downloadvmusic-7bb77279bf5b76902c167923d57800e1f6f6073e.tar.gz
vmusic-7bb77279bf5b76902c167923d57800e1f6f6073e.zip
Cleaning up Adlib module
Diffstat (limited to 'pcmalsa.cpp')
-rw-r--r--pcmalsa.cpp42
1 files changed, 21 insertions, 21 deletions
diff --git a/pcmalsa.cpp b/pcmalsa.cpp
index eec9b94..5708509 100644
--- a/pcmalsa.cpp
+++ b/pcmalsa.cpp
@@ -36,13 +36,13 @@ PCMOutAlsa::~PCMOutAlsa()
int PCMOutAlsa::open(const char *dev, unsigned int sampleRate, unsigned int channels)
{
- int err;
+ int err;
err = snd_pcm_open(&_pcm, dev, SND_PCM_STREAM_PLAYBACK, 0);
if (err < 0) {
- LogWarn(("ALSA playback open error: %s\n", snd_strerror(err)));
- return VERR_AUDIO_STREAM_COULD_NOT_CREATE;
- }
+ LogWarn(("ALSA playback open error: %s\n", snd_strerror(err)));
+ return VERR_AUDIO_STREAM_COULD_NOT_CREATE;
+ }
// TODO: Right now, setting a too large period size means we will not let the main
// thread run long enough to actually change notes/voices. Need some actual synchronization.
@@ -57,16 +57,16 @@ int PCMOutAlsa::open(const char *dev, unsigned int sampleRate, unsigned int chan
return VERR_AUDIO_STREAM_COULD_NOT_CREATE;
}
- return VINF_SUCCESS;
+ return VINF_SUCCESS;
}
int PCMOutAlsa::close()
{
- if (_pcm) {
+ if (_pcm) {
snd_pcm_drain(_pcm);
- snd_pcm_close(_pcm);
- }
- return VINF_SUCCESS;
+ snd_pcm_close(_pcm);
+ }
+ return VINF_SUCCESS;
}
ssize_t PCMOutAlsa::avail()
@@ -104,16 +104,16 @@ int PCMOutAlsa::wait()
ssize_t PCMOutAlsa::write(int16_t *buf, size_t n)
{
- snd_pcm_sframes_t frames = snd_pcm_writei(_pcm, buf, n);
- if (frames < 0) {
- LogFlow(("ALSA trying to recover from error: %s\n", snd_strerror(frames)));
- frames = snd_pcm_recover(_pcm, frames, 0);
- }
- if (frames < 0) {
+ snd_pcm_sframes_t frames = snd_pcm_writei(_pcm, buf, n);
+ if (frames < 0) {
+ LogFlow(("ALSA trying to recover from error: %s\n", snd_strerror(frames)));
+ frames = snd_pcm_recover(_pcm, frames, 0);
+ }
+ if (frames < 0) {
LogWarn(("ALSA write error: %s\n", snd_strerror(frames)));
- return VERR_AUDIO_STREAM_NOT_READY;
- }
- return frames;
+ return VERR_AUDIO_STREAM_NOT_READY;
+ }
+ return frames;
}
int PCMOutAlsa::setParams(unsigned int sampleRate, unsigned int channels, unsigned int bufferTime, unsigned int periodTime)
@@ -175,19 +175,19 @@ int PCMOutAlsa::setParams(unsigned int sampleRate, unsigned int channels, unsign
}
err = snd_pcm_hw_params_get_buffer_size(hwparams, &_bufferSize);
if (err < 0) {
- printf("Unable to get buffer size for playback: %s\n", snd_strerror(err));
+ LogWarnFunc(("Unable to get buffer size for playback: %s\n", snd_strerror(err)));
return err;
}
/* set the period time */
err = snd_pcm_hw_params_set_period_time_near(_pcm, hwparams, &periodTime, NULL);
if (err < 0) {
- printf("Unable to set period time %u for playback: %s\n", periodTime, snd_strerror(err));
+ LogWarnFunc(("Unable to set period time %u for playback: %s\n", periodTime, snd_strerror(err)));
return err;
}
err = snd_pcm_hw_params_get_period_size(hwparams, &_periodSize, NULL);
if (err < 0) {
- printf("Unable to get period size for playback: %s\n", snd_strerror(err));
+ LogWarnFunc(("Unable to get period size for playback: %s\n", snd_strerror(err)));
return err;
}