From 15769ead725b7c215dedd4ea5196955086d2044a Mon Sep 17 00:00:00 2001

From: Lennart Poettering <mznyfn@0pointer.de>

Date: Fri, 18 Jul 2008 21:22:50 +0200

Subject: [PATCH] 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 <lennart@poettering.net>

Signed-off-by: Takashi Iwai <tiwai@suse.de>

---

 src/pcm/pcm_plugin.c |    6 +++---

 1 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/pcm/pcm_plugin.c b/src/pcm/pcm_plugin.c

index c199d8d..b377cb2 100644

--- a/src/pcm/pcm_plugin.c

+++ b/src/pcm/pcm_plugin.c

@@ -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;

 	}

-- 

1.5.5.1