![[tokkee]](http://tokkee.org/images/avatar.png){kind=avatar}
diff --git a/src/strfsong.c b/src/strfsong.c
index ed3e20f9807ad56fa1c91621a1f551dfd96f879f..b64c350ba6dace76d83b8c70f0dd49d591b1eefc 100644 (file)
--- a/src/strfsong.c
+++ b/src/strfsong.c
-/*
- * Based on mpc's songToFormatedString modified for glib and ncmpc
- *
- *
- * (c) 2003-2004 by normalperson and Warren Dukes (shank@mercury.chem.pitt.edu)
- * and Daniel Brown (danb@cs.utexas.edu)
- * and Kalle Wallin (kaw@linux.se)
+/* ncmpc (Ncurses MPD Client)
+ * (c) 2004-2017 The Music Player Daemon Project
+ * Project homepage: http://musicpd.org
*
* This program is free software; you can redistribute it and/or modify
* it under the terms of the GNU General Public License as published by
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
* GNU General Public License for more details.
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
+ *
+ * You should have received a copy of the GNU General Public License along
+ * with this program; if not, write to the Free Software Foundation, Inc.,
+ * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
*/
#include "strfsong.h"
#include "charset.h"
+#include "time_format.h"
+
+#include <mpd/client.h>
#include <string.h>
return p;
}
+#ifndef NCMPC_MINI
+
+static char *
+concat_tag_values(const char *a, const char *b)
+{
+ return g_strconcat(a, ", ", b, NULL);
+}
+
+static char *
+song_more_tag_values(const struct mpd_song *song, enum mpd_tag_type tag,
+ const char *first)
+{
+ const char *p = mpd_song_get_tag(song, tag, 1);
+ if (p == NULL)
+ return NULL;
+
+ char *buffer = concat_tag_values(first, p);
+ for (unsigned i = 2; (p = mpd_song_get_tag(song, tag, i)) != NULL;
+ ++i) {
+ char *prev = buffer;
+ buffer = concat_tag_values(buffer, p);
+ g_free(prev);
+ }
+
+ return buffer;
+}
+
+#endif /* !NCMPC_MINI */
+
+static char *
+song_tag_locale(const struct mpd_song *song, enum mpd_tag_type tag)
+{
+ const char *value = mpd_song_get_tag(song, tag, 0);
+ if (value == NULL)
+ return NULL;
+
+#ifndef NCMPC_MINI
+ char *all = song_more_tag_values(song, tag, value);
+ if (all != NULL)
+ value = all;
+#endif /* !NCMPC_MINI */
+
+ char *result = utf8_to_locale(value);
+
+#ifndef NCMPC_MINI
+ g_free(all);
+#endif /* !NCMPC_MINI */
+
+ return result;
+}
+
static gsize
_strfsong(gchar *s,
gsize max,
const struct mpd_song *song,
const gchar **last)
{
- const gchar *p, *end;
- gchar *temp;
- gsize n, length = 0;
- gboolean found = FALSE;
+ bool found = false;
+ /* "missed" helps handling the case of mere literal text like
+ found==true instead of found==false. */
+ bool missed = false;
+
+ s[0] = '\0';
- memset(s, 0, max);
if (song == NULL)
return 0;
+ const char *p;
+ size_t length = 0;
for (p = format; *p != '\0' && length<max;) {
/* OR */
if (p[0] == '|') {
++p;
- if(!found) {
- memset(s, 0, max);
+ if(missed && !found) {
+ s[0] = '\0';
length = 0;
+ missed = false;
} else {
p = skip(p);
}
/* AND */
if (p[0] == '&') {
++p;
- if(!found) {
+ if(missed && !found) {
p = skip(p);
} else {
- found = FALSE;
+ found = false;
+ missed = false;
}
continue;
}
/* EXPRESSION START */
if (p[0] == '[') {
- temp = g_malloc0(max);
+ char *temp = g_malloc0(max);
if( _strfsong(temp, max, p+1, song, &p) >0 ) {
g_strlcat(s, temp, max);
length = strlen(s);
- found = TRUE;
+ found = true;
+ } else {
+ missed = true;
}
g_free(temp);
continue;
/* EXPRESSION END */
if (p[0] == ']') {
if(last) *last = p+1;
- if(!found && length) {
- memset(s, 0, max);
+ if(missed && !found && length) {
+ s[0] = '\0';
length = 0;
}
return length;
/* pass-through non-escaped portions of the format string */
if (p[0] != '#' && p[0] != '%' && length<max) {
s[length++] = *p;
+ s[length] = '\0';
p++;
continue;
}
/* let the escape character escape itself */
if (p[0] == '#' && p[1] != '\0' && length<max) {
s[length++] = *(p+1);
+ s[length] = '\0';
p+=2;
continue;
}
/* advance past the esc character */
/* find the extent of this format specifier (stop at \0, ' ', or esc) */
- temp = NULL;
- end = p+1;
+ char *temp = NULL;
+ const char *end = p + 1;
while(*end >= 'a' && *end <= 'z') {
end++;
}
- n = end - p + 1;
+ size_t n = end - p + 1;
if(*end != '%')
n--;
else if (strncmp("%file%", p, n) == 0)
- temp = utf8_to_locale(song->file);
- else if (strncmp("%artist%", p, n) == 0)
- temp = song->artist ? utf8_to_locale(song->artist) : NULL;
- else if (strncmp("%title%", p, n) == 0)
- temp = song->title ? utf8_to_locale(song->title) : NULL;
- else if (strncmp("%album%", p, n) == 0)
- temp = song->album ? utf8_to_locale(song->album) : NULL;
+ temp = utf8_to_locale(mpd_song_get_uri(song));
+ else if (strncmp("%artist%", p, n) == 0) {
+ temp = song_tag_locale(song, MPD_TAG_ARTIST);
+ if (temp == NULL) {
+ temp = song_tag_locale(song, MPD_TAG_PERFORMER);
+ if (temp == NULL)
+ temp = song_tag_locale(song, MPD_TAG_COMPOSER);
+ }
+ } else if (strncmp("%albumartist", p, n) == 0)
+ temp = song_tag_locale(song, MPD_TAG_ALBUM_ARTIST);
+ else if (strncmp("%composer%", p, n) == 0)
+ temp = song_tag_locale(song, MPD_TAG_COMPOSER);
+ else if (strncmp("%performer%", p, n) == 0)
+ temp = song_tag_locale(song, MPD_TAG_PERFORMER);
+ else if (strncmp("%title%", p, n) == 0) {
+ temp = song_tag_locale(song, MPD_TAG_TITLE);
+ if (temp == NULL)
+ temp = song_tag_locale(song, MPD_TAG_NAME);
+ } else if (strncmp("%album%", p, n) == 0)
+ temp = song_tag_locale(song, MPD_TAG_ALBUM);
else if (strncmp("%shortalbum%", p, n) == 0) {
- temp = song->album ? utf8_to_locale(song->album) : NULL;
+ temp = song_tag_locale(song, MPD_TAG_ALBUM);
if (temp) {
gchar *temp2 = g_strndup(temp, 25);
if (strlen(temp) > 25) {
}
}
else if (strncmp("%track%", p, n) == 0)
- temp = song->track ? utf8_to_locale(song->track) : NULL;
+ temp = song_tag_locale(song, MPD_TAG_TRACK);
+ else if (strncmp("%disc%", p, n) == 0)
+ temp = song_tag_locale(song, MPD_TAG_DISC);
else if (strncmp("%name%", p, n) == 0)
- temp = song->name ? utf8_to_locale(song->name) : NULL;
+ temp = song_tag_locale(song, MPD_TAG_NAME);
else if (strncmp("%date%", p, n) == 0)
- temp = song->date ? utf8_to_locale(song->date) : NULL;
+ temp = song_tag_locale(song, MPD_TAG_DATE);
else if (strncmp("%genre%", p, n) == 0)
- temp = song->genre ? utf8_to_locale(song->genre) : NULL;
+ temp = song_tag_locale(song, MPD_TAG_GENRE);
else if (strncmp("%shortfile%", p, n) == 0) {
- if( strstr(song->file, "://") )
- temp = utf8_to_locale(song->file);
+ const char *uri = mpd_song_get_uri(song);
+ if (strstr(uri, "://") != NULL)
+ temp = utf8_to_locale(uri);
else
- temp = utf8_to_locale(g_basename(song->file));
+ temp = utf8_to_locale(g_basename(uri));
} else if (strncmp("%time%", p, n) == 0) {
- if (song->time != MPD_SONG_NO_TIME) {
- if (song->time > 3600) {
- temp = g_strdup_printf("%d:%02d:%02d",
- song->time / 3600,
- (song->time % 3600) / 60,
- song->time % 60);
- } else {
- temp = g_strdup_printf("%d:%02d",
- song->time / 60,
- song->time % 60);
- }
+ unsigned duration = mpd_song_get_duration(song);
+
+ if (duration > 0) {
+ char buffer[32];
+ format_duration_short(buffer, sizeof(buffer),
+ duration);
+ temp = g_strdup(buffer);
}
}
if( temp == NULL) {
gsize templen=n;
/* just pass-through any unknown specifiers (including esc) */
- /* drop a null char in so printf stops at the end of this specifier,
- but put the real character back in (pseudo-const) */
if( length+templen > max )
templen = max-length;
- g_strlcat(s, p,max);
+ gchar *ident = g_strndup(p, templen);
+ g_strlcat(s, ident, max);
length+=templen;
+ g_free(ident);
+
+ missed = true;
} else {
gsize templen = strlen(temp);
- found = TRUE;
+ found = true;
if( length+templen > max )
templen = max-length;
g_strlcat(s, temp, max);