diff options
author | Jaroslav Kysela <perex@perex.cz> | 2014-10-01 15:43:57 +0200 |
---|---|---|
committer | Jaroslav Kysela <perex@perex.cz> | 2014-10-01 15:43:57 +0200 |
commit | 8f361d83cfcb39887f5fc591633e68d9448e3425 (patch) | |
tree | 25642de9190b7c1bc62eb697cbbcb936aa761eb5 | |
parent | 89a4f490bd974e454d82942100f241bb1a2270f3 (diff) | |
download | alsa-utils-8f361d83cfcb39887f5fc591633e68d9448e3425.tar.gz |
Revert "aplay: fix pcm_read() return value"
This reverts commit 8aa13eec80eac312e4b99423909387660fb99b8f.
The semantics for pcm_read() and pcm_readv() was changed, but the
callers expect the exact frame count as requested. It's possible
to fix callers, but the fix is more complicated than to revert the
change. Note that '-d' processing was broken in some cases.
Note: The reverted commit allows that the return value might be
greater than requested (see the first condition in read routines).
-rw-r--r-- | aplay/aplay.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/aplay/aplay.c b/aplay/aplay.c index 30d3f31..e58e1bc 100644 --- a/aplay/aplay.c +++ b/aplay/aplay.c @@ -2039,7 +2039,7 @@ static ssize_t pcm_read(u_char *data, size_t rcount) data += r * bits_per_frame / 8; } } - return result; + return rcount; } static ssize_t pcm_readv(u_char **data, unsigned int channels, size_t rcount) @@ -2084,7 +2084,7 @@ static ssize_t pcm_readv(u_char **data, unsigned int channels, size_t rcount) count -= r; } } - return result; + return rcount; } /* |