Code

screen_lyrics: put code for reloading in a function
[ncmpc.git] / src / screen_lyrics.c
index 15f5721ec1bfcbf293a13df3ff0ed285dae953c6..a8c721068542563a57713d8c1503733b0e07fac5 100644 (file)
@@ -1,10 +1,6 @@
-/* 
- * $Id: screen_lyrics.c 3355 2006-09-1 17:44:04Z tradiaz $
- *     
- * (c) 2006 by Kalle Wallin <kaw@linux.se>
- * Tue Aug  1 23:17:38 2006
- * lyrics enhancement written by Andreas Obergrusberger <tradiaz@yahoo.de> 
- * using www.leoslyrics.com XML API
+/* ncmpc (Ncurses MPD Client)
+ * (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
  * 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 "screen_lyrics.h"
+#include "screen_interface.h"
+#include "screen_status.h"
+#include "screen_file.h"
+#include "screen_song.h"
+#include "i18n.h"
+#include "options.h"
+#include "mpdclient.h"
+#include "screen.h"
+#include "lyrics.h"
+#include "screen_text.h"
+
+#include <assert.h>
+#include <sys/stat.h>
 #include <stdlib.h>
 #include <string.h>
 #include <glib.h>
-#include <ncurses.h>
-#include <expat.h>
 #include <unistd.h>
+#include <stdio.h>
 
-#include "config.h"
-#ifndef DISABLE_LYRICS_SCREEN
-#include "ncmpc.h"
-#include "options.h"
-#include "mpdclient.h"
-#include "command.h"
-#include "screen.h"
-#include "screen_utils.h"
-#include "easy_download.h"
-#include "strfsong.h"
+static struct screen_text text;
 
-#define LEOSLYRICS_SEARCH_URL "http://api.leoslyrics.com/api_search.php?auth=QuodLibet&artist=%s&songtitle=%s"
-#define LEOSLYRICS_CONTENT_URL "http://api.leoslyrics.com/api_lyrics.php?auth=QuodLibet&hid=%s"
+static struct mpd_song *next_song;
+static bool follow = false;
 
-#define CREDITS "Lyrics provided by www.LeosLyrics.com"
-typedef struct _formed_text
-{
-       GString *text;
-       GArray *lines;
-       int val;
-} formed_text;
-
-
-XML_Parser parser, contentp;
-static int lyrics_text_rows = -1;
-static list_window_t *lw = NULL;
-guint8 result;
-char *hid;
-GTimer *dltime;
-short int lock;
-//GString *lyr_text;
-//char *lyr_text;      
-//GArray *textline;
-formed_text lyr_text;
-/* result is a bitset in which the succes when searching 4 lyrics is logged
-countend by position - backwards
-0: lyrics in database
-1: proper access  to the lyrics provider
-2: lyrics found
-3: exact match
-4: lyrics downloaded
-5: lyrics saved
-wasting 3 bits doesn't mean being a fat memory hog like kde.... does it?
-*/
-static void lyrics_paint(screen_t *screen, mpdclient_t *c);
-
-void get_text_line(formed_text *text, int num, char *dest, int len)
+static struct {
+       struct mpd_song *song;
+
+       char *artist, *title, *plugin_name;
+
+       struct plugin_cycle *loader;
+
+       guint loader_timeout;
+} current;
+
+static void
+screen_lyrics_abort(void)
 {
-       memset(dest, '\0', len*sizeof(char));
-    if(num >= text->lines->len-1) return;      
-       int linelen;
-       if(num == 0)
-       {
-               linelen = g_array_index(text->lines, int, num);
-               memcpy(dest, text->text->str, linelen*sizeof(char));    
+       if (current.loader != NULL) {
+               plugin_stop(current.loader);
+               current.loader = NULL;
        }
-       else if(num == 1)
-       { //dont ask me why, but this is needed....
-               linelen = g_array_index(text->lines, int, num)
-                       - g_array_index(text->lines, int, num-1);
-               memcpy(dest, &text->text->str[g_array_index(text->lines, int, num-1)],
-               linelen*sizeof(char));  
-       }       
-       else
-       {
-               linelen = g_array_index(text->lines, int, num+1)
-                       - g_array_index(text->lines, int, num);
-               memcpy(dest, &text->text->str[g_array_index(text->lines, int, num)],
-               linelen*sizeof(char));  
+
+       if (current.loader_timeout != 0) {
+               g_source_remove(current.loader_timeout);
+               current.loader_timeout = 0;
        }
-       dest[linelen] = '\n';
-       dest[linelen+1] = '\0';
-}
-       
-void add_text_line(formed_text *dest, const char *src, int len)
-{
-       // need this because g_array_append_val doesnt work with literals
-       // and expat sends "\n" as an extra line everytime
-       if(len == 0)
-       {
-               dest->val = strlen(src);
-               if(dest->lines->len > 0) dest->val += g_array_index(dest->lines, int,
-                                                                       dest->lines->len-1);
-               g_string_append(dest->text, src);
-               g_array_append_val(dest->lines, dest->val);
-               return;
+
+       if (current.plugin_name != NULL) {
+               g_free(current.plugin_name);
+               current.plugin_name = NULL;
        }
-       if(len > 1 || dest->val == 0) 
-       {
-               dest->val = len;        
-               if(dest->lines->len > 0) dest->val += g_array_index(dest->lines, int,
-                                                                       dest->lines->len-1);
+
+       if (current.artist != NULL) {
+               g_free(current.artist);
+               current.artist = NULL;
        }
-       else if (len == 1 && dest->val != 0) dest->val = 0;
-                               
-       if(dest->val > 0)
-       { 
-               g_string_append_len(dest->text, src, len);
-               g_array_append_val(dest->lines, dest->val);
+
+       if (current.title != NULL) {
+               g_free(current.title);
+               current.title = NULL;
+       }
+
+       if (current.song != NULL) {
+               mpd_song_free(current.song);
+               current.song = NULL;
        }
 }
 
-void formed_text_init(formed_text *text)
+/**
+ * Repaint and update the screen, if it is currently active.
+ */
+static void
+lyrics_repaint_if_active(void)
 {
-       if(text->text != NULL) g_string_free(text->text, TRUE); 
-       text->text = g_string_new("");
-       
-       if(text->lines != NULL) g_array_free(text->lines, TRUE);
-       text->lines = g_array_new(FALSE, TRUE, 4);
-       
-       text->val = 0;
+       if (screen_is_visible(&screen_lyrics)) {
+               screen_text_repaint(&text);
+
+               /* XXX repaint the screen title */
+       }
 }
-/*
-char *check_lyr_hd(char *artist, char *title, int how)
-{ //checking whether for lyrics file existence and proper access
-       static char path[1024];
-       snprintf(path, 1024, "%s/.lyrics/%s/%s.lyric", 
+
+static void
+path_lyr_file(char *path, size_t size,
+               const char *artist, const char *title)
+{
+       snprintf(path, size, "%s/.lyrics/%s - %s.txt",
                        getenv("HOME"), artist, title);
-    
-    if(g_access(path, how) != 0) return NULL;
-       return path;
-}              
+}
 
-int get_lyr_hd(char *artist, char *title)
+static bool
+exists_lyr_file(const char *artist, const char *title)
 {
-       char *path = check_lyr_hd(artist, title, R_OK);
-       if(path == NULL) return -1;
-       
-       FILE *lyr_file; 
-       lyr_file = fopen(path, "r");
-       if(lyr_file == NULL) return -1;
-       
-       char *buf = NULL;
-       char **line = &buf;
-       size_t n = 0;
-       
-       while(1)
-       {
-        n = getline(line, &n, lyr_file); 
-        if( n < 1 || *line == NULL || feof(lyr_file) != 0 ) return 0;
-        add_text_line(&lyr_text, *line, n+1);
-        free(*line);
-        *line = NULL; n = 0;
-       }
-       
-       return 0;
-}      
-    
-int store_lyr_hd()
+       char path[1024];
+       struct stat result;
+
+       path_lyr_file(path, 1024, artist, title);
+
+       return (stat(path, &result) == 0);
+}
+
+static FILE *
+create_lyr_file(const char *artist, const char *title)
+{
+       char path[1024];
+
+       snprintf(path, 1024, "%s/.lyrics",
+                getenv("HOME"));
+       mkdir(path, S_IRWXU);
+
+       path_lyr_file(path, 1024, artist, title);
+
+       return fopen(path, "w");
+}
+
+static int
+store_lyr_hd(void)
 {
-       char artist[512];
-       char title[512];
-       static char path[1024];
        FILE *lyr_file;
-       
-       get_text_line(&lyr_text, 0, artist, 512);
-       get_text_line(&lyr_text, 1, title, 512);
-       artist[strlen(artist)-2] = '\0';
-       title[strlen(title)-2] = '\0';
-       
-       snprintf(path, 1024, "%s/.lyrics/%s/%s.lyric", 
-                       getenv("HOME"), artist, title);
-       
-       if (check_lyr_hd(artist, title, F_OK) == 0) lyr_file = fopen(path, "w");
-       else lyr_file = fopen(path, "xw");
-       if(lyr_file == NULL) return -1;
-       
-       int i;
-       char line_buf[1024];
-       
-       for(i = 2; i <= lyr_text.text->len; i++)
-       {
-               get_text_line(&lyr_text, i, line_buf, 0);
-               fputs(line_buf, lyr_file);
-       }
+       unsigned i;
+
+       lyr_file = create_lyr_file(current.artist, current.title);
+       if (lyr_file == NULL)
+               return -1;
+
+       for (i = 0; i < text.lines->len; ++i)
+               fprintf(lyr_file, "%s\n",
+                       (const char*)g_ptr_array_index(text.lines, i));
+
        fclose(lyr_file);
        return 0;
 }
-                               
-*/     
-void check_repaint()
-{
-       if(screen_get_id("lyrics") == get_cur_mode_id())lyrics_paint(NULL, NULL);
-}
 
-int check_dl_progress(void *clientp, double dltotal, double dlnow,
-                        double ultotal, double ulnow)
+static int
+delete_lyr_hd(void)
 {
-       if(g_timer_elapsed(dltime, NULL) >= options.lyrics_timeout)
-       {       
-               formed_text_init(&lyr_text);
+       char path[1024];
+
+       if (!exists_lyr_file(current.artist, current.title))
                return -1;
-       }
-       return 0;
-}      
 
+       path_lyr_file(path, 1024, current.artist, current.title);
+       if (unlink(path) != 0)
+               return -2;
 
+       return 0;
+}
 
-static void check_content(void *data, const char *name, const char **atts)
+static void
+screen_lyrics_set(const GString *str)
 {
-       if(strstr(name, "text") != NULL)
-       {
+       screen_text_set(&text, str);
 
-               result |= 16;
-       }
+       /* paint new data */
+
+       lyrics_repaint_if_active();
 }
-       
 
-static void check_search_response(void *data, const char *name,
-                const char **atts)
+static void
+screen_lyrics_callback(const GString *result, const bool success,
+                      const char *plugin_name, G_GNUC_UNUSED void *data)
 {
-       if(strstr(name, "response") != NULL)
-       {
-       result |=2;
-       return;
-       }  
-       
-       if(result & 4)
-       {
-               if(strstr(name, "result") != NULL)
-               {
-                       if(strstr(atts[2], "hid") != NULL)
-                       {
-                               hid = atts[3];
-                       }
-       
-                       if(strstr(atts[2], "exactMatch") != NULL)
-                       {
-                               result |= 8;
-                       }                       
-               }
+       assert(current.loader != NULL);
+
+       current.plugin_name = g_strdup(plugin_name);
+
+       /* Display result, which may be lyrics or error messages */
+       if (result != NULL)
+               screen_lyrics_set(result);
+
+       if (success == true) {
+               if (options.lyrics_autosave &&
+                   !exists_lyr_file(current.artist, current.title))
+                       store_lyr_hd();
+       } else {
+               /* translators: no lyrics were found for the song */
+               screen_status_message (_("No lyrics"));
+       }
+
+       if (current.loader_timeout != 0) {
+               g_source_remove(current.loader_timeout);
+               current.loader_timeout = 0;
        }
-                       
+
+       plugin_stop(current.loader);
+       current.loader = NULL;
 }
 
-static void end_tag(void *data, const char *name)
+static gboolean
+screen_lyrics_timeout_callback(gpointer G_GNUC_UNUSED data)
 {
-  //hmmmmmm            
-}
+       plugin_stop(current.loader);
+       current.loader = NULL;
 
-  static void check_search_success(void *userData, const XML_Char *s, int len)
-    {
-       if(result & 2)  //lets first check whether we're right
-       {               //we don't really want to search in the wrong string
-               if(strstr((char*) s, "SUCCESS"))
-               {
-               result |=4;
-               }
-       }       
-    }
+       screen_status_printf(_("Lyrics timeout occurred after %d seconds"),
+                            options.lyrics_timeout);
 
-static void fetch_text(void *userData, const XML_Char *s, int len) 
+       current.loader_timeout = 0;
+       return FALSE;
+}
+
+static void
+screen_lyrics_load(const struct mpd_song *song)
 {
-       if(result & 16)
-       {
-               add_text_line(&lyr_text, s, len); 
+       const char *artist, *title;
+
+       assert(song != NULL);
+
+       screen_lyrics_abort();
+       screen_text_clear(&text);
+
+       artist = mpd_song_get_tag(song, MPD_TAG_ARTIST, 0);
+       title = mpd_song_get_tag(song, MPD_TAG_TITLE, 0);
+
+       current.song = mpd_song_dup(song);
+       current.artist = g_strdup(artist);
+       current.title = g_strdup(title);
+
+       current.loader = lyrics_load(current.artist, current.title,
+                                    screen_lyrics_callback, NULL);
+
+       if (options.lyrics_timeout != 0) {
+               current.loader_timeout =
+                       g_timeout_add_seconds(options.lyrics_timeout,
+                                             screen_lyrics_timeout_callback,
+                                             NULL);
        }
 }
 
-gpointer get_lyr(void *c)
+static void
+screen_lyrics_reload(void)
 {
-       mpd_Status *status = ((mpdclient_t*)c)->status;
-       mpd_Song *cur = ((mpdclient_t*)c)->song; 
-       mpdclient_update((mpdclient_t*)c);
-       
-       if(!(IS_PAUSED(status->state)||IS_PLAYING(status->state)))
-       {
-               formed_text_init(&lyr_text);                    
-               return NULL;
+       if (current.loader == NULL && current.artist != NULL &&
+           current.title != NULL) {
+               current.loader = lyrics_load(current.artist, current.title,
+                                            screen_lyrics_callback, NULL);
+               screen_text_repaint(&text);
        }
-       
-       char url_avail[256];
-       char url_hid[256];
-       char artist[MAX_SONGNAME_LENGTH];
-       char title[MAX_SONGNAME_LENGTH];
-       lock = 2;
-       result = 0;
-       
-       if(dltime == NULL) dltime = g_timer_new();
-       
-       strfsong(artist, MAX_SONGNAME_LENGTH, "%artist%", cur);
-       strfsong(title, MAX_SONGNAME_LENGTH, "%title%", cur);
-       
-       formed_text_init(&lyr_text);
-       add_text_line(&lyr_text, artist, 0);
-       add_text_line(&lyr_text, title, 0);
-       
-       //if(get_lyr_hd(artist, title) == 0) return &lyr_text;
-       
-       //this replacess the whitespaces with '+'
-       g_strdelimit(artist, " ", '+');
-       g_strdelimit(title, " ", '+');
-       
-       //we insert the artist and the title into the url               
-       snprintf(url_avail, 512, LEOSLYRICS_SEARCH_URL, artist, title);
-
-       //download that xml!
-       easy_download_struct lyr_avail = {NULL, 0,-1};  
-       
-       g_timer_start(dltime);
-       if(!easy_download(url_avail, &lyr_avail, check_dl_progress)) return NULL;
-       g_timer_stop(dltime);
-
-       //we gotta parse that stuff with expat
-       parser = XML_ParserCreate(NULL);
-        XML_SetUserData(parser, NULL);
-       
-       int state = 0;
-
-       XML_SetElementHandler(parser, check_search_response, end_tag);
-       XML_SetCharacterDataHandler(parser, check_search_success);
-       XML_Parse(parser, lyr_avail.data, strlen(lyr_avail.data), state);       
-       XML_ParserFree(parser); 
-
-       if(!(result & 4)) return NULL; //check whether lyrics found
-       easy_download_struct lyr_content = {NULL, 0,-1};  
-       snprintf(url_hid, 512, LEOSLYRICS_CONTENT_URL, hid);
-       
-       g_timer_continue(dltime);               
-       if(!(easy_download(url_hid, &lyr_content, check_dl_progress))) return NULL;
-       g_timer_stop(dltime);
-       
-       contentp = XML_ParserCreate(NULL);
-       XML_SetUserData(contentp, NULL);
-       XML_SetElementHandler(contentp, check_content, end_tag);        
-       XML_SetCharacterDataHandler(contentp, fetch_text);
-       XML_Parse(contentp, lyr_content.data, strlen(lyr_content.data), state);
-       XML_ParserFree(contentp);
-       
-       lw->start = 0;
-       check_repaint();
-       
-    lock = 1;
-       return &lyr_text;
-}      
-
-static char *
-list_callback(int index, int *highlight, void *data)
-{
-       static char buf[512];
-       int linelen;
-
-    //i think i'ts fine to write it into the 1st line...
-  if((index == lyr_text.lines->len && lyr_text.lines->len != 2)||
-         ((lyr_text.lines->len == 0 
-         ||lyr_text.lines->len == 2) && index == 0))
-  {
-    *highlight=3; 
-       return CREDITS;
-  }
-    
-  if(index < 2 && lyr_text.lines->len > 2) *highlight=3;
-  else if(index >=  lyr_text.lines->len || index == 2)
-  {
-         return "";
-  }
-  if(index >1) index--;
-  get_text_line(&lyr_text, index, buf, 512);
-  return buf;
-} 
-
+}
 
 static void
-lyrics_init(WINDOW *w, int cols, int rows)
+lyrics_screen_init(WINDOW *w, int cols, int rows)
 {
-  lw = list_window_init(w, cols, rows);
-  lw->flags = LW_HIDE_CURSOR;
-  //lyr_text.lines = g_array_new(FALSE, TRUE, 4);
-  formed_text_init(&lyr_text);
-  if (!g_thread_supported()) g_thread_init(NULL);
-  
+       screen_text_init(&text, w, cols, rows);
 }
 
 static void
 lyrics_resize(int cols, int rows)
 {
-  lw->cols = cols;
-  lw->rows = rows;
+       screen_text_resize(&text, cols, rows);
 }
 
 static void
 lyrics_exit(void)
 {
-  list_window_free(lw);
-}
+       screen_lyrics_abort();
 
+       screen_text_deinit(&text);
+}
 
-static char *
-lyrics_title(char *str, size_t size)
+static void
+lyrics_open(struct mpdclient *c)
 {
-       if(lyr_text.lines->len == 2){ 
-       if(lock == 1 && !(result & 2)) return _("Lyrics  [No connection]");
-    if(lock == 1 && !(result & 4)) return _("Lyrics  [Not found]");            
-       if(lock == 2) return _("Lyrics  [retrieving]");
+       const struct mpd_song *next_song_c =
+               next_song != NULL ? next_song : c->song;
+
+       if (next_song_c != NULL &&
+           (current.song == NULL ||
+            strcmp(mpd_song_get_uri(next_song_c),
+                   mpd_song_get_uri(current.song)) != 0))
+               screen_lyrics_load(next_song_c);
+
+       if (next_song != NULL) {
+               mpd_song_free(next_song);
+               next_song = NULL;
        }
-       /*if(lyr_text.lines->len > 2) 
-       {
-               static char buf[512];
-               char artist[512];
-               char title[512];
-               get_text_line(&lyr_text, 0, artist, 512);
-               get_text_line(&lyr_text, 1, artist, 512);
-               snprintf(buf, 512, "Lyrics  %s - %s", artist, title);
-               return buf;
-       }*/
-       return "Lyrics";
 }
 
-static void 
-lyrics_paint(screen_t *screen, mpdclient_t *c)
+static void
+lyrics_update(struct mpdclient *c)
 {
-  lw->clear = 1;
-  list_window_paint(lw, list_callback, NULL);
-  wrefresh(lw->w);
+       if (!follow)
+               return;
+
+       if (c->song != NULL &&
+           (current.song == NULL ||
+            strcmp(mpd_song_get_uri(c->song),
+                   mpd_song_get_uri(current.song)) != 0))
+               screen_lyrics_load(c->song);
 }
 
-static void 
-lyrics_update(screen_t *screen, mpdclient_t *c)
-{  
-  if( lw->repaint )
-    {
-      list_window_paint(lw, list_callback, NULL);
-      wrefresh(lw->w);
-      lw->repaint = 0;
-    }
+static const char *
+lyrics_title(char *str, size_t size)
+{
+       if (current.loader != NULL) {
+               snprintf(str, size, "%s (%s)",
+                        _("Lyrics"),
+                        /* translators: this message is displayed
+                           while data is retrieved */
+                        _("loading..."));
+               return str;
+       } else if (current.artist != NULL && current.title != NULL &&
+                  !screen_text_is_empty(&text)) {
+               int n;
+               n = snprintf(str, size, "%s: %s - %s",
+                            _("Lyrics"),
+                            current.artist, current.title);
+
+               if (options.lyrics_show_plugin && current.plugin_name != NULL &&
+                   (unsigned int) n < size - 1)
+                       snprintf(str + n, size - n, " (%s)",
+                                current.plugin_name);
+
+               return str;
+       } else
+               return _("Lyrics");
 }
 
+static void
+lyrics_paint(void)
+{
+       screen_text_paint(&text);
+}
 
-static int 
-lyrics_cmd(screen_t *screen, mpdclient_t *c, command_t cmd)
+static bool
+lyrics_cmd(struct mpdclient *c, command_t cmd)
 {
-  lw->repaint=1;
-  switch(cmd)
-    {
-    case CMD_LIST_NEXT:
-      if( lw->start+lw->rows < lyr_text.lines->len+1 )
-       lw->start++;
-      return 1;
-    case CMD_LIST_PREVIOUS:
-      if( lw->start >0 )
-       lw->start--;
-      return 1;
-    case CMD_LIST_FIRST:
-      lw->start = 0;
-      return 1;
-    case CMD_LIST_LAST:
-      lw->start = lyrics_text_rows-lw->rows;
-      if( lw->start<0 )
-       lw->start = 0;
-      return 1;
-    case CMD_LIST_NEXT_PAGE:
-      lw->start = lw->start + lw->rows-1;
-      if( lw->start+lw->rows >= lyr_text.lines->len+1 )
-       lw->start = lyr_text.lines->len-lw->rows+1;
-      if( lw->start<0 )
-       lw->start = 0;
-       return 1;
-    case CMD_LIST_PREVIOUS_PAGE:
-      lw->start = lw->start - lw->rows;
-      if( lw->start<0 )
-       lw->start = 0;
-      return 1;
+       if (screen_text_cmd(&text, c, cmd))
+               return true;
+
+       switch(cmd) {
+       case CMD_INTERRUPT:
+               if (current.loader != NULL) {
+                       screen_lyrics_abort();
+                       screen_text_clear(&text);
+               }
+               return true;
+       case CMD_SAVE_PLAYLIST:
+               if (current.loader == NULL && current.artist != NULL &&
+                   current.title != NULL && store_lyr_hd() == 0)
+                       /* lyrics for the song were saved on hard disk */
+                       screen_status_message (_("Lyrics saved"));
+               return true;
+       case CMD_DELETE:
+               if (current.loader == NULL && current.artist != NULL &&
+                   current.title != NULL) {
+                       switch (delete_lyr_hd()) {
+                       case 0:
+                               screen_status_message (_("Lyrics deleted"));
+                               break;
+                       case -1:
+                               screen_status_message (_("No saved lyrics"));
+                               break;
+                       }
+               }
+               return true;
+       case CMD_LYRICS_UPDATE:
+               if (c->song != NULL) {
+                       screen_lyrics_load(c->song);
+                       screen_text_repaint(&text);
+               }
+               return true;
        case CMD_SELECT:
-         g_thread_create(get_lyr, c, FALSE, NULL);     
-         return 1;     
+               screen_lyrics_reload();
+               return true;
+
+#ifdef ENABLE_SONG_SCREEN
+       case CMD_SCREEN_SONG:
+               if (current.song != NULL) {
+                       screen_song_switch(c, current.song);
+                       return true;
+               }
+
+               break;
+#endif
+       case CMD_SCREEN_SWAP:
+               screen_swap(c, current.song);
+               return true;
+
+       case CMD_LOCATE:
+               if (current.song != NULL) {
+                       screen_file_goto_song(c, current.song);
+                       return true;
+               }
+
+               return false;
+
        default:
-      break;
-    }
-
-  lw->selected = lw->start+lw->rows;
-  if( screen_find(screen, c, 
-                 lw,  lyrics_text_rows,
-                 cmd, list_callback, NULL) )
-    {
-      /* center the row */
-      lw->start = lw->selected-(lw->rows/2);
-      if( lw->start+lw->rows > lyrics_text_rows )
-       lw->start = lyrics_text_rows-lw->rows;
-      if( lw->start<0 )
-       lw->start=0;
-      return 1;
-    }
-
-  return 0;
-}
+               break;
+       }
 
-static list_window_t *
-lyrics_lw(void)
-{
-  return lw;
+       return false;
 }
 
-screen_functions_t *
-get_screen_lyrics(void)
+const struct screen_functions screen_lyrics = {
+       .init = lyrics_screen_init,
+       .exit = lyrics_exit,
+       .open = lyrics_open,
+       .update = lyrics_update,
+       .close = NULL,
+       .resize = lyrics_resize,
+       .paint = lyrics_paint,
+       .cmd = lyrics_cmd,
+       .get_title = lyrics_title,
+};
+
+void
+screen_lyrics_switch(struct mpdclient *c, const struct mpd_song *song, bool f)
 {
-  static screen_functions_t functions;
-
-  memset(&functions, 0, sizeof(screen_functions_t));
-  functions.init   = lyrics_init;
-  functions.exit   = lyrics_exit;
-  functions.open   = NULL;
-  functions.close  = NULL;
-  functions.resize = lyrics_resize;
-  functions.paint  = lyrics_paint;
-  functions.update = lyrics_update;
-  functions.cmd    = lyrics_cmd;
-  functions.get_lw = lyrics_lw;
-  functions.get_title = lyrics_title;
-
-  return &functions;
+       assert(song != NULL);
+
+       follow = f;
+       next_song = mpd_song_dup(song);
+       screen_switch(&screen_lyrics, c);
 }
-#endif /* ENABLE_LYRICS_SCREEN */