At Fri, 15 Oct 2010 12:35:31 +0200,
David Henningsson wrote:
>
> I'm using speaker-test in an Ubuntu-specific bug triaging
> script/application ("ubuntu-bug audio"), and without showing the
> terminal that speaker-test outputs. So when the stream cannot be opened
> for one reason or another, my application hangs indefinitely and the
> user cannot report the bug.
>
> Looking at the code, seems like Takashi removed all error loops except
> this one four years ago, so there shouldn't be any controversy in
> removing this one as well.
Looks good to me.
Applied now. Thanks!
Takashi
>
> --
> David Henningsson, Canonical Ltd.
> http://launchpad.net/~diwic
> [2 0001-speaker-test-Don-t-retry-after-fatal-errors.patch <text/x-patch (7bit)>]
> >From f51771c4064ec5f18ec46aeec2d5fe294a4a372c Mon Sep 17 00:00:00 2001
> From: David Henningsson <david.henningsson@xxxxxxxxxxxxx>
> Date: Fri, 15 Oct 2010 12:30:44 +0200
> Subject: [PATCH] speaker-test: Don't retry after fatal errors
>
> Fixup commit 9b1a2566: Remove error loop
>
> Signed-off-by: David Henningsson <david.henningsson@xxxxxxxxxxxxx>
> ---
> speaker-test/speaker-test.c | 4 ++--
> 1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/speaker-test/speaker-test.c b/speaker-test/speaker-test.c
> index 458a8d7..3029110 100644
> --- a/speaker-test/speaker-test.c
> +++ b/speaker-test/speaker-test.c
> @@ -993,9 +993,9 @@ int main(int argc, char *argv[]) {
>
> }
>
> - while ((err = snd_pcm_open(&handle, device, SND_PCM_STREAM_PLAYBACK, 0)) < 0) {
> + if ((err = snd_pcm_open(&handle, device, SND_PCM_STREAM_PLAYBACK, 0)) < 0) {
> printf(_("Playback open error: %d,%s\n"), err,snd_strerror(err));
> - sleep(1);
> + exit(EXIT_FAILURE);
> }
>
> if ((err = set_hwparams(handle, hwparams, SND_PCM_ACCESS_RW_INTERLEAVED)) < 0) {
> --
> 1.7.1
>
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
