[pulseaudio-discuss] [PATCH] cli-command: Report error in pa_play_file.

[Tanu Kaskinen]

On Tue, 2018-05-29 at 09:22 +0200, Tomaz Solc wrote:
> Current code does not check whether pa_play_file call failed. Hence no error is
> reported in the cli interface if playback failed because e.g. file isn't
> readable by the daemon.
> ---
>  src/pulsecore/cli-command.c | 7 ++++++-
>  1 file changed, 6 insertions(+), 1 deletion(-)
> 
> diff --git a/src/pulsecore/cli-command.c b/src/pulsecore/cli-command.c
> index 44795b0d..db09e5dc 100644
> --- a/src/pulsecore/cli-command.c
> +++ b/src/pulsecore/cli-command.c
> @@ -1282,7 +1282,12 @@ static int pa_cli_command_play_file(pa_core *c, pa_tokenizer *t, pa_strbuf *buf,
>          return -1;
>      }
>  
> -    return pa_play_file(sink, fname, NULL);
> +    if (pa_play_file(sink, fname, NULL)) {
> +        pa_strbuf_puts(buf, "Failed to play sound file.\n");
> +        return -1;
> +    }
> +
> +    return 0;
>  }
>  
>  static int pa_cli_command_list_shared_props(pa_core *c, pa_tokenizer *t, pa_strbuf *buf, bool *fail) {

Thanks! Applied to the "next" branch.

-- 
Tanu

https://liberapay.com/tanuk
https://www.patreon.com/tanuk