Skip to contentSkip to conversation

fix type of internally used sframes variable, to avoid unnecessary casts

PATCH New Lennart Poettering 1 participant 1 message Jul 18, 2008

lore
Share & tools
lore

Lennart Poettering

author
#0 lore raw

PATCH 2/3 fix type of internally used sframes variable, to avoid unnecessary casts

This minor patch fixes the type of the sframes variable in snd_pcm_plugin_forward(). With this fix we need to cast less and the code is less confusing.

Signed-off-by Lennart Poettering <[email protected]>
---
 src/pcm/pcm_plugin.c |    6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)
src/pcm/pcm_plugin.c +3 -3
index c199d8d..b377cb2 100644
@@ -234,7 +234,7 @@ static snd_pcm_sframes_t snd_pcm_plugin_forward(snd_pcm_t *pcm, snd_pcm_uframes_
 {
 	snd_pcm_plugin_t *plugin = pcm->private_data;
 	snd_pcm_sframes_t n = snd_pcm_mmap_avail(pcm);
-	snd_pcm_uframes_t sframes;
+	snd_pcm_sframes_t sframes;
 
 	if ((snd_pcm_uframes_t)n < frames)
 		frames = n;

@@ -246,8 +246,8 @@ static snd_pcm_sframes_t snd_pcm_plugin_forward(snd_pcm_t *pcm, snd_pcm_uframes_
 	else
 		sframes = frames;
 	snd_atomic_write_begin(&plugin->watom);
-	sframes = INTERNAL(snd_pcm_forward)(plugin->gen.slave, (snd_pcm_uframes_t) sframes);
-	if ((snd_pcm_sframes_t) sframes < 0) {
+	sframes = INTERNAL(snd_pcm_forward)(plugin->gen.slave, sframes);
+	if (sframes < 0) {
 		snd_atomic_write_end(&plugin->watom);
 		return sframes;
 	}
Signature · 8 lines
1.5.6

Lennart

-- 
Lennart Poettering                        Red Hat, Inc.
lennart [at] poettering [dot] net         ICQ# 11060553
http://0pointer.net/lennart/           GnuPG 0x1A015CC4