![[tokkee]](http://tokkee.org/images/avatar.png){kind=avatar}
diff --git a/src/screen_browser.c b/src/screen_browser.c
index 6375ee955325c13b7206b0003b568834b62c2b53..49827647d6e8f0898032fbad629c3b69b6e3a6e9 100644 (file)
--- a/src/screen_browser.c
+++ b/src/screen_browser.c
/* ncmpc (Ncurses MPD Client)
- * (c) 2004-2009 The Music Player Daemon Project
+ * (c) 2004-2010 The Music Player Daemon Project
* Project homepage: http://musicpd.org
* This program is free software; you can redistribute it and/or modify
#include "strfsong.h"
#include "mpdclient.h"
#include "filelist.h"
+#include "colors.h"
+#include "paint.h"
+#include "song_paint.h"
#include <mpd/client.h>
#endif
/* list_window callback */
-const char *
-browser_lw_callback(unsigned idx, bool *highlight, G_GNUC_UNUSED char **second_column, void *data)
+static const char *
+browser_lw_callback(unsigned idx, void *data)
{
const struct filelist *fl = (const struct filelist *) data;
static char buf[BUFSIZE];
const struct filelist_entry *entry;
const struct mpd_entity *entity;
- if (fl == NULL || idx >= filelist_length(fl))
- return NULL;
+ assert(fl != NULL);
+ assert(idx < filelist_length(fl));
entry = filelist_get(fl, idx);
assert(entry != NULL);
entity = entry->entity;
-#ifndef NCMPC_MINI
- *highlight = (entry->flags & HIGHLIGHT) != 0;
-#else
- *highlight = false;
-#endif
if( entity == NULL )
- return "[..]";
+ return "..";
if (mpd_entity_get_type(entity) == MPD_ENTITY_TYPE_DIRECTORY) {
const struct mpd_directory *dir =
mpd_entity_get_directory(entity);
char *directory = utf8_to_locale(g_basename(mpd_directory_get_path(dir)));
-
- g_snprintf(buf, BUFSIZE, "[%s]", directory);
+ g_strlcpy(buf, directory, sizeof(buf));
g_free(directory);
return buf;
} else if (mpd_entity_get_type(entity) == MPD_ENTITY_TYPE_SONG) {
load_playlist(struct mpdclient *c, const struct mpd_playlist *playlist)
{
struct mpd_connection *connection = mpdclient_get_connection(c);
- char *filename = utf8_to_locale(mpd_playlist_get_path(playlist));
+
+ if (connection == NULL)
+ return false;
if (mpd_run_load(connection, mpd_playlist_get_path(playlist))) {
+ char *filename = utf8_to_locale(mpd_playlist_get_path(playlist));
screen_status_printf(_("Loading playlist %s..."),
g_basename(filename));
+ g_free(filename);
+
c->events |= MPD_IDLE_QUEUE;
} else
mpdclient_handle_error(c);
- g_free(filename);
return true;
}
struct filelist_entry *
browser_get_selected_entry(const struct screen_browser *browser)
{
+ struct list_window_range range;
+
+ list_window_get_range(browser->lw, &range);
+
if (browser->filelist == NULL ||
- browser->lw->selected_start < browser->lw->selected_end ||
- browser->lw->selected >= filelist_length(browser->filelist))
+ range.end <= range.start ||
+ range.end > range.start + 1 ||
+ range.start >= filelist_length(browser->filelist))
return NULL;
- return filelist_get(browser->filelist, browser->lw->selected);
+ return filelist_get(browser->filelist, range.start);
}
static const struct mpd_entity *
static bool
browser_handle_select(struct screen_browser *browser, struct mpdclient *c)
{
+ struct list_window_range range;
struct filelist_entry *entry;
+ bool success = false;
- if (browser->lw->range_selection) {
- for (unsigned i = browser->lw->selected_start;
- i <= browser->lw->selected_end; i++) {
- entry = browser_get_index(browser, i);
+ list_window_get_range(browser->lw, &range);
+ for (unsigned i = range.start; i < range.end; ++i) {
+ entry = browser_get_index(browser, i);
- if (entry != NULL && entry->entity != NULL)
- browser_select_entry(c, entry, TRUE);
- }
- return false;
- } else {
- entry = browser_get_selected_entry(browser);
-
- if (entry == NULL || entry->entity == NULL)
- return false;
-
- return browser_select_entry(c, entry, TRUE);
+ if (entry != NULL && entry->entity != NULL)
+ success = browser_select_entry(c, entry, TRUE);
}
+
+ return range.end == range.start + 1 && success;
}
static bool
browser_handle_add(struct screen_browser *browser, struct mpdclient *c)
{
+ struct list_window_range range;
struct filelist_entry *entry;
+ bool success = false;
- if (browser->lw->range_selection) {
- for (unsigned i = browser->lw->selected_start;
- i <= browser->lw->selected_end; i++) {
- entry = browser_get_index(browser, i);
-
- if (entry != NULL && entry->entity != NULL)
- browser_select_entry(c, entry, FALSE);
- }
- return false;
- } else {
- entry = browser_get_selected_entry(browser);
+ list_window_get_range(browser->lw, &range);
+ for (unsigned i = range.start; i < range.end; ++i) {
+ entry = browser_get_index(browser, i);
- if (entry == NULL || entry->entity == NULL)
- return false;
-
- return browser_select_entry(c, entry, FALSE);
+ if (entry != NULL && entry->entity != NULL)
+ success = browser_select_entry(c, entry, FALSE) ||
+ success;
}
+
+ return range.end == range.start + 1 && success;
}
static void
@@ -372,19 +367,12 @@ browser_handle_mouse_event(struct screen_browser *browser, struct mpdclient *c)
int row;
unsigned prev_selected = browser->lw->selected;
unsigned long bstate;
- int length;
-
- if (browser->filelist)
- length = filelist_length(browser->filelist);
- else
- length = 0;
if (screen_get_mouse_event(c, &bstate, &row) ||
- list_window_mouse(browser->lw, length, bstate, row))
+ list_window_mouse(browser->lw, bstate, row))
return 1;
- browser->lw->selected = browser->lw->start + row;
- list_window_check_selected(browser->lw, length);
+ list_window_set_cursor(browser->lw, browser->lw->start + row);
if( bstate & BUTTON1_CLICKED ) {
if (prev_selected == browser->lw->selected)
@@ -398,6 +386,10 @@ browser_handle_mouse_event(struct screen_browser *browser, struct mpdclient *c)
}
#endif
+static void
+screen_browser_paint_callback(WINDOW *w, unsigned i, unsigned y,
+ unsigned width, bool selected, void *data);
+
bool
browser_cmd(struct screen_browser *browser,
struct mpdclient *c, command_t cmd)
if (browser->filelist == NULL)
return false;
- if (list_window_cmd(browser->lw, filelist_length(browser->filelist),
- cmd))
+ if (list_window_cmd(browser->lw, cmd))
return true;
switch (cmd) {
case CMD_LIST_RFIND:
case CMD_LIST_FIND_NEXT:
case CMD_LIST_RFIND_NEXT:
- screen_find(browser->lw, filelist_length(browser->filelist),
- cmd, browser_lw_callback,
+ screen_find(browser->lw, cmd, browser_lw_callback,
browser->filelist);
return true;
case CMD_LIST_JUMP:
- screen_jump(browser->lw, browser_lw_callback, browser->filelist);
+ screen_jump(browser->lw, browser_lw_callback,
+ screen_browser_paint_callback, browser->filelist);
return true;
#ifdef HAVE_GETMOUSE
case CMD_SELECT:
if (browser_handle_select(browser, c))
- list_window_cmd(browser->lw,
- filelist_length(browser->filelist),
- CMD_LIST_NEXT);
+ list_window_cmd(browser->lw, CMD_LIST_NEXT);
return true;
case CMD_ADD:
if (browser_handle_add(browser, c))
- list_window_cmd(browser->lw,
- filelist_length(browser->filelist),
- CMD_LIST_NEXT);
+ list_window_cmd(browser->lw, CMD_LIST_NEXT);
return true;
case CMD_SELECT_ALL:
return false;
}
+
+void
+screen_browser_paint_directory(WINDOW *w, unsigned width,
+ bool selected, const char *name)
+{
+ row_color(w, COLOR_DIRECTORY, selected);
+
+ waddch(w, '[');
+ waddstr(w, name);
+ waddch(w, ']');
+
+ /* erase the unused space after the text */
+ row_clear_to_eol(w, width, selected);
+}
+
+static void
+screen_browser_paint_playlist(WINDOW *w, unsigned width,
+ bool selected, const char *name)
+{
+ row_paint_text(w, width, COLOR_PLAYLIST, selected, name);
+}
+
+static void
+screen_browser_paint_callback(WINDOW *w, unsigned i,
+ unsigned y, unsigned width,
+ bool selected, void *data)
+{
+ const struct filelist *fl = (const struct filelist *) data;
+ const struct filelist_entry *entry;
+ const struct mpd_entity *entity;
+ bool highlight;
+ const struct mpd_directory *directory;
+ const struct mpd_playlist *playlist;
+ char *p;
+
+ assert(fl != NULL);
+ assert(i < filelist_length(fl));
+
+ entry = filelist_get(fl, i);
+ assert(entry != NULL);
+
+ entity = entry->entity;
+ if (entity == NULL) {
+ screen_browser_paint_directory(w, width, selected, "..");
+ return;
+ }
+
+#ifndef NCMPC_MINI
+ highlight = (entry->flags & HIGHLIGHT) != 0;
+#else
+ highlight = false;
+#endif
+
+ switch (mpd_entity_get_type(entity)) {
+ case MPD_ENTITY_TYPE_DIRECTORY:
+ directory = mpd_entity_get_directory(entity);
+ p = utf8_to_locale(g_basename(mpd_directory_get_path(directory)));
+ screen_browser_paint_directory(w, width, selected, p);
+ g_free(p);
+ break;
+
+ case MPD_ENTITY_TYPE_SONG:
+ paint_song_row(w, y, width, selected, highlight,
+ mpd_entity_get_song(entity), NULL);
+ break;
+
+ case MPD_ENTITY_TYPE_PLAYLIST:
+ playlist = mpd_entity_get_playlist(entity);
+ p = utf8_to_locale(g_basename(mpd_playlist_get_path(playlist)));
+ screen_browser_paint_playlist(w, width, selected, p);
+ g_free(p);
+ break;
+
+ default:
+ row_paint_text(w, width, highlight ? COLOR_LIST_BOLD : COLOR_LIST,
+ selected, "<unknown>");
+ }
+}
+
+void
+screen_browser_paint(const struct screen_browser *browser)
+{
+ list_window_paint2(browser->lw, screen_browser_paint_callback,
+ browser->filelist);
+}