diff options
author | Lars-Dominik Braun <lars@6xq.net> | 2024-03-29 10:55:31 +0100 |
---|---|---|
committer | Lars-Dominik Braun <lars@6xq.net> | 2024-03-29 10:55:31 +0100 |
commit | ee735ee68792ab93af5c66368e046a77eec3fc6e (patch) | |
tree | dc81c852f16e4720bea078350658d2eadbfab320 /src/ui_act.c | |
parent | c76bcb27d7864f33e0d64ca4afbb2453cabfdb79 (diff) | |
download | pianobar-ee735ee68792ab93af5c66368e046a77eec3fc6e.tar.gz pianobar-ee735ee68792ab93af5c66368e046a77eec3fc6e.tar.bz2 pianobar-ee735ee68792ab93af5c66368e046a77eec3fc6e.zip |
piano: Do not return empty explanation.
Instead return NULL and let the client handle it.
Diffstat (limited to 'src/ui_act.c')
-rw-r--r-- | src/ui_act.c | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/src/ui_act.c b/src/ui_act.c index 30bd4e5..fa5c43b 100644 --- a/src/ui_act.c +++ b/src/ui_act.c @@ -271,8 +271,12 @@ BarUiActCallback(BarUiActExplain) { BarUiMsg (&app->settings, MSG_INFO, "Receiving explanation... "); if (BarUiActDefaultPianoCall (PIANO_REQUEST_EXPLAIN, &reqData)) { - BarUiMsg (&app->settings, MSG_INFO, "%s\n", reqData.retExplain); - free (reqData.retExplain); + if (reqData.retExplain == NULL) { + BarUiMsg (&app->settings, MSG_ERR, "No explanation provided.\n"); + } else { + BarUiMsg (&app->settings, MSG_INFO, "%s\n", reqData.retExplain); + free (reqData.retExplain); + } } BarUiActDefaultEventcmd ("songexplain"); } |