![[tokkee]](http://tokkee.org/images/avatar.png){kind=avatar}
diff --git a/src/screen_artist.c b/src/screen_artist.c
index 5ff6f9b81c8d258128fab32783c5da9b1de7b95a..f8d50d0c096def9358075320d970d75811e12992 100644 (file)
--- a/src/screen_artist.c
+++ b/src/screen_artist.c
/* ncmpc (Ncurses MPD Client)
- * (c) 2004-2010 The Music Player Daemon Project
+ * (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
#include "mpdclient.h"
#include "screen_browser.h"
#include "filelist.h"
+#include "options.h"
#include <assert.h>
#include <string.h>
compare_utf8(gconstpointer s1, gconstpointer s2)
{
const char *const*t1 = s1, *const*t2 = s2;
- char *key1, *key2;
- int n;
- key1 = g_utf8_collate_key(*t1,-1);
- key2 = g_utf8_collate_key(*t2,-1);
- n = strcmp(key1,key2);
+ char *key1 = g_utf8_collate_key(*t1,-1);
+ char *key2 = g_utf8_collate_key(*t2,-1);
+ int n = strcmp(key1,key2);
g_free(key1);
g_free(key2);
return n;
screen_artist_lw_callback(unsigned idx, void *data)
{
GPtrArray *list = data;
- static char buf[BUFSIZE];
- char *str, *str_utf8;
if (mode == LIST_ALBUMS) {
if (idx == 0)
assert(idx < list->len);
- str_utf8 = g_ptr_array_index(list, idx);
+ char *str_utf8 = g_ptr_array_index(list, idx);
assert(str_utf8 != NULL);
- str = utf8_to_locale(str_utf8);
+ char *str = utf8_to_locale(str_utf8);
+
+ static char buf[BUFSIZE];
g_strlcpy(buf, str, sizeof(buf));
g_free(str);
return buf;
}
-static void
-screen_artist_paint(void);
-
-static void
-artist_repaint(void)
-{
- screen_artist_paint();
- wrefresh(browser.lw->w);
-}
-
static void
string_array_free(GPtrArray *array)
{
- unsigned i;
-
- for (i = 0; i < array->len; ++i) {
+ for (unsigned i = 0; i < array->len; ++i) {
char *value = g_ptr_array_index(array, i);
g_free(value);
}
screen_artist_init(WINDOW *w, int cols, int rows)
{
browser.lw = list_window_init(w, cols, rows);
+ browser.song_format = options.list_format;
artist = NULL;
album = NULL;
}
static void
paint_artist_callback(WINDOW *w, unsigned i,
gcc_unused unsigned y, unsigned width,
- bool selected, void *data)
+ bool selected, const void *data)
{
- GPtrArray *list = data;
+ const GPtrArray *list = data;
char *p = utf8_to_locale(g_ptr_array_index(list, i));
screen_browser_paint_directory(w, width, selected, p);
static void
paint_album_callback(WINDOW *w, unsigned i,
gcc_unused unsigned y, unsigned width,
- bool selected, void *data)
+ bool selected, const void *data)
{
- GPtrArray *list = data;
+ const GPtrArray *list = data;
const char *p;
char *q = NULL;
static const char *
screen_artist_get_title(char *str, size_t size)
{
- char *s1, *s2;
-
switch(mode) {
+ char *s1, *s2;
+
case LIST_ARTISTS:
g_snprintf(str, size, _("All artists"));
break;
| MPD_IDLE_QUEUE
#endif
))
- artist_repaint();
+ screen_artist_paint();
}
/* add_query - Add all songs satisfying specified criteria.
const char *_artist)
{
struct mpd_connection *connection = mpdclient_get_connection(c);
- char *str;
- struct filelist *addlist;
assert(_filter != NULL);
if (connection == NULL)
return;
- str = utf8_to_locale(_filter);
+ char *str = utf8_to_locale(_filter);
if (table == MPD_TAG_ALBUM)
screen_status_printf(_("Adding album %s..."), str);
else
MPD_TAG_ARTIST, _artist);
mpd_search_commit(connection);
- addlist = filelist_new_recv(connection);
+ struct filelist *addlist = filelist_new_recv(connection);
if (mpdclient_finish_command(c))
mpdclient_filelist_add_all(c, addlist);
static bool
screen_artist_cmd(struct mpdclient *c, command_t cmd)
{
- struct list_window_range range;
- char *selected;
- char *old;
- char *old_ptr;
- int idx;
-
switch(cmd) {
+ struct list_window_range range;
+ char *selected;
+ char *old;
+ char *old_ptr;
+ int idx;
+
case CMD_PLAY:
switch (mode) {
case LIST_ARTISTS:
open_album_list(c, g_strdup(selected));
list_window_reset(browser.lw);
- artist_repaint();
+ screen_artist_paint();
return true;
case LIST_ALBUMS:
list_window_reset(browser.lw);
}
- artist_repaint();
+ screen_artist_paint();
return true;
case LIST_SONGS:
list_window_center(browser.lw, idx);
}
- artist_repaint();
+ screen_artist_paint();
return true;
}
break;
break;
}
- artist_repaint();
+ screen_artist_paint();
break;
case CMD_GO_ROOT_DIRECTORY:
break;
}
- artist_repaint();
+ screen_artist_paint();
break;
case CMD_SELECT:
case LIST_ARTISTS:
screen_find(browser.lw, cmd,
screen_artist_lw_callback, artist_list);
- artist_repaint();
+ screen_artist_paint();
return true;
case LIST_ALBUMS:
screen_find(browser.lw, cmd,
screen_artist_lw_callback, album_list);
- artist_repaint();
+ screen_artist_paint();
return true;
case LIST_SONGS:
case CMD_LIST_JUMP:
switch (mode) {
case LIST_ARTISTS:
- screen_jump(browser.lw, screen_artist_lw_callback,
+ screen_jump(browser.lw,
+ screen_artist_lw_callback, artist_list,
paint_artist_callback, artist_list);
- artist_repaint();
+ screen_artist_paint();
return true;
case LIST_ALBUMS:
- screen_jump(browser.lw, screen_artist_lw_callback,
+ screen_jump(browser.lw,
+ screen_artist_lw_callback, album_list,
paint_album_callback, album_list);
- artist_repaint();
+ screen_artist_paint();
return true;
case LIST_SONGS:
if (screen_artist_lw_cmd(c, cmd)) {
if (screen_is_visible(&screen_artist))
- artist_repaint();
+ screen_artist_paint();
return true;
}