From 34637e024a702d5ff5a431f14d67e2a7fb6331c4 Mon Sep 17 00:00:00 2001 From: Kalle Wallin Date: Tue, 14 Jun 2005 17:30:17 +0000 Subject: [PATCH] Moved list window state code to list_window.c git-svn-id: https://svn.musicpd.org/ncmpc/trunk@3353 09075e82-0dd4-0310-85a5-a0d7c8717e4f --- ChangeLog | 4 ++ src/list_window.c | 59 +++++++++++++++++++++++++ src/list_window.h | 13 ++++++ src/screen_artist.c | 103 +++++++++++++++++++++----------------------- src/screen_file.c | 49 +++------------------ 5 files changed, 132 insertions(+), 96 deletions(-) diff --git a/ChangeLog b/ChangeLog index 964bfcb..393bb68 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,5 +1,9 @@ +2005-06-14: Kalle Wallin + * Moved list window state code to list_window.c + 2005-06-11: Kalle Wallin * configure.ac: support wide-char ncurses library (--with-ncursesw) + * support.c: added my_strlen for wide-char strings 2005-06-06: Kalle Wallin * Added Norwegian translations from Ole R. Thorsen diff --git a/src/list_window.c b/src/list_window.c index 8966622..5ee81d0 100644 --- a/src/list_window.c +++ b/src/list_window.c @@ -327,3 +327,62 @@ list_window_cmd(list_window_t *lw, int rows, command_t cmd) } + + + +list_window_state_t * +list_window_init_state(void) +{ + return g_malloc0(sizeof(list_window_state_t)); +} + +list_window_state_t * +list_window_free_state(list_window_state_t *state) +{ + if( state ) + { + if( state->list ) + { + GList *list = state->list; + while( list ) + { + g_free(list->data); + list->data = NULL; + list = list->next; + } + g_list_free(state->list); + state->list = NULL; + } + g_free(state); + } + return NULL; +} + +void +list_window_push_state(list_window_state_t *state, list_window_t *lw) +{ + if( state ) + { + list_window_t *tmp = g_malloc(sizeof(list_window_t)); + memcpy(tmp, lw, sizeof(list_window_t)); + state->list = g_list_prepend(state->list, (gpointer) tmp); + list_window_reset(lw); + } +} + +void +list_window_pop_state(list_window_state_t *state, list_window_t *lw) +{ + if( state && state->list ) + { + list_window_t *tmp = state->list->data; + + memcpy(lw, tmp, sizeof(list_window_t)); + g_free(tmp); + state->list->data = NULL; + state->list = g_list_delete_link(state->list, state->list); + } +} + + + diff --git a/src/list_window.h b/src/list_window.h index 8374e53..2061e81 100644 --- a/src/list_window.h +++ b/src/list_window.h @@ -23,6 +23,11 @@ typedef struct } list_window_t; +typedef struct +{ + GList *list; +} list_window_state_t; + /* create a new list window */ list_window_t *list_window_init(WINDOW *w, int width, int height); @@ -71,4 +76,12 @@ list_window_rfind(list_window_t *lw, int wrap, int rows); +/* list window states */ +list_window_state_t *list_window_init_state(void); +list_window_state_t *list_window_free_state(list_window_state_t *state); +void list_window_push_state(list_window_state_t *state, list_window_t *lw); +void list_window_pop_state(list_window_state_t *state, list_window_t *lw); + + + #endif diff --git a/src/screen_artist.c b/src/screen_artist.c index e09d9f0..a363304 100644 --- a/src/screen_artist.c +++ b/src/screen_artist.c @@ -1,7 +1,7 @@ /* * $Id$ * - * (c) 2004 by Kalle Wallin + * (c) 2005 by Kalle Wallin * * 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 @@ -45,32 +45,7 @@ static list_window_t *lw = NULL; static mpdclient_filelist_t *filelist = NULL; static int metalist_length = 0; static GList *metalist = NULL; -static GList *lw_state_list = NULL; - -/* store current state when entering a subdirectory */ -static void -push_lw_state(void) -{ - list_window_t *tmp = g_malloc(sizeof(list_window_t)); - - memcpy(tmp, lw, sizeof(list_window_t)); - lw_state_list = g_list_prepend(lw_state_list, (gpointer) tmp); -} - -/* get previous state when leaving a directory */ -static void -pop_lw_state(void) -{ - if( lw_state_list ) - { - list_window_t *tmp = lw_state_list->data; - - memcpy(lw, tmp, sizeof(list_window_t)); - g_free(tmp); - lw_state_list->data = NULL; - lw_state_list = g_list_delete_link(lw_state_list, lw_state_list); - } -} +static list_window_state_t *lw_state = NULL; /* list_window callback */ static char * @@ -108,6 +83,7 @@ playlist_changed_callback(mpdclient_t *c, int event, gpointer data) } } +/* fetch artists/albums/songs from mpd */ static void update_metalist(mpdclient_t *c, char *m_artist, char *m_album) { @@ -160,12 +136,31 @@ update_metalist(mpdclient_t *c, char *m_artist, char *m_album) metalist = mpdclient_get_artists_utf8(c); } metalist_length = g_list_length(metalist); + lw->clear = TRUE; +} + +/* db updated */ +static void +browse_callback(mpdclient_t *c, int event, gpointer data) +{ + switch(event) + { + case BROWSE_DB_UPDATED: + D("screen_artist.c> browse_callback() [BROWSE_DB_UPDATED]\n"); + lw->clear = 1; + lw->repaint = 1; + update_metalist(c, g_strdup(artist), g_strdup(album)); + break; + default: + break; + } } static void init(WINDOW *w, int cols, int rows) { lw = list_window_init(w, cols, rows); + lw_state = list_window_init_state(); artist = NULL; album = NULL; } @@ -173,20 +168,6 @@ init(WINDOW *w, int cols, int rows) static void quit(void) { - /* clear list window state */ - if( lw_state_list ) - { - GList *list = lw_state_list; - while( list ) - { - g_free(list->data); - list->data = NULL; - list = list->next; - } - g_list_free(lw_state_list); - lw_state_list = NULL; - } - if( filelist ) filelist = mpdclient_filelist_free(filelist); if( metalist ) @@ -195,14 +176,22 @@ quit(void) g_free(album); artist = NULL; album = NULL; - list_window_free(lw); + lw = list_window_free(lw); + lw_state = list_window_free_state(lw_state); } static void open(screen_t *screen, mpdclient_t *c) { + static gboolean callback_installed = FALSE; + if( metalist==NULL && filelist ==NULL) update_metalist(c, NULL, NULL); + if( !callback_installed ) + { + mpdclient_install_browse_callback(c, browse_callback); + callback_installed = TRUE; + } } static void @@ -242,12 +231,18 @@ paint(screen_t *screen, mpdclient_t *c) static void update(screen_t *screen, mpdclient_t *c) { - if( filelist==NULL || filelist->updated ) + if( filelist && !filelist->updated ) + { + list_window_paint(lw, browse_lw_callback, (void *) filelist); + } + else if( metalist ) + { + list_window_paint(lw, artist_lw_callback, (void *) metalist); + } + else { paint(screen, c); - return; } - list_window_paint(lw, browse_lw_callback, (void *) filelist); wnoutrefresh(lw->w); } @@ -287,7 +282,7 @@ artist_cmd(screen_t *screen, mpdclient_t *c, command_t cmd) update_metalist(c, g_strdup(artist), NULL); list_window_reset(lw); /* restore previous list window state */ - pop_lw_state(); + list_window_pop_state(lw_state,lw); } else browse_handle_enter(screen, c, lw, filelist); @@ -300,28 +295,25 @@ artist_cmd(screen_t *screen, mpdclient_t *c, command_t cmd) update_metalist(c, NULL, NULL); list_window_reset(lw); /* restore previous list window state */ - pop_lw_state(); + list_window_pop_state(lw_state,lw); } else if( lw->selected == metalist_length-1) /* handle "show all" */ { update_metalist(c, g_strdup(artist), g_strdup("\0")); - push_lw_state(); - list_window_reset(lw); + list_window_push_state(lw_state,lw); } else /* select album */ { char *selected = (char *) g_list_nth_data(metalist, lw->selected); update_metalist(c, g_strdup(artist), g_strdup(selected)); - push_lw_state(); - list_window_reset(lw); + list_window_push_state(lw_state,lw); } } else { char *selected = (char *) g_list_nth_data(metalist, lw->selected); update_metalist(c, g_strdup(selected), NULL); - push_lw_state(); - list_window_reset(lw); + list_window_push_state(lw_state,lw); } return 1; @@ -335,6 +327,11 @@ artist_cmd(screen_t *screen, mpdclient_t *c, command_t cmd) /* continue and update... */ case CMD_SCREEN_UPDATE: + screen->painted = 0; + lw->clear = 1; + lw->repaint = 1; + update_metalist(c, g_strdup(artist), g_strdup(album)); + screen_status_printf(_("Screen updated!")); return 0; case CMD_LIST_FIND: diff --git a/src/screen_file.c b/src/screen_file.c index 002e9cd..b2ca1b6 100644 --- a/src/screen_file.c +++ b/src/screen_file.c @@ -45,7 +45,7 @@ static list_window_t *lw = NULL; -static GList *lw_state_list = NULL; +static list_window_state_t *lw_state = NULL; static mpdclient_filelist_t *filelist = NULL; @@ -153,31 +153,6 @@ playlist_changed_callback(mpdclient_t *c, int event, gpointer data) } } -/* store current state when entering a subdirectory */ -static void -push_lw_state(void) -{ - list_window_t *tmp = g_malloc(sizeof(list_window_t)); - - memcpy(tmp, lw, sizeof(list_window_t)); - lw_state_list = g_list_prepend(lw_state_list, (gpointer) tmp); -} - -/* get previous state when leaving a directory */ -static void -pop_lw_state(void) -{ - if( lw_state_list ) - { - list_window_t *tmp = lw_state_list->data; - - memcpy(lw, tmp, sizeof(list_window_t)); - g_free(tmp); - lw_state_list->data = NULL; - lw_state_list = g_list_delete_link(lw_state_list, lw_state_list); - } -} - /* list_window callback */ char * browse_lw_callback(int index, int *highlight, void *data) @@ -248,7 +223,7 @@ change_directory(screen_t *screen, mpdclient_t *c, filelist_entry_t *entry) path = g_strdup(parent); list_window_reset(lw); /* restore previous list window state */ - pop_lw_state(); + list_window_pop_state(lw_state,lw); } else if( entity->type==MPD_INFO_ENTITY_TYPE_DIRECTORY) @@ -257,8 +232,7 @@ change_directory(screen_t *screen, mpdclient_t *c, filelist_entry_t *entry) mpd_Directory *dir = entity->info.directory; path = utf8_to_locale(dir->path); /* save current list window state */ - push_lw_state(); - list_window_reset(lw); + list_window_push_state(lw_state,lw); } else return -1; @@ -537,6 +511,7 @@ static void browse_init(WINDOW *w, int cols, int rows) { lw = list_window_init(w, cols, rows); + lw_state = list_window_init_state(); } static void @@ -549,22 +524,10 @@ browse_resize(int cols, int rows) static void browse_exit(void) { - if( lw_state_list ) - { - GList *list = lw_state_list; - while( list ) - { - g_free(list->data); - list->data = NULL; - list = list->next; - } - g_list_free(lw_state_list); - lw_state_list = NULL; - - } if( filelist ) filelist = mpdclient_filelist_free(filelist); - list_window_free(lw); + lw = list_window_free(lw); + lw_state = list_window_free_state(lw_state); } static void -- 2.30.2