screen_queue: update list_window length after mouse delete

[ncmpc.git] / src / screen_file.c

diff --git a/src/screen_file.c b/src/screen_file.c
index c5688fe6263cd520a8005a298141dd1d0896fb8e..67dc3e081182da8addac2436b3b77b54103b1e8b 100644 (file)
--- a/src/screen_file.c
+++ b/src/screen_file.c
@@ -21,6 +21,7 @@
 #include "screen_browser.h"
 #include "screen_interface.h"
 #include "screen_message.h"
+#include "screen_queue.h"
 #include "screen.h"
 #include "config.h"
 #include "i18n.h"
@@ -28,7 +29,6 @@
 #include "mpdclient.h"
 #include "filelist.h"
 #include "screen_utils.h"
-#include "screen_play.h"
 #include "screen_client.h"
 
 #include <mpd/client.h>
@@ -52,10 +52,26 @@ screen_file_repaint(void)
 }
 
 static void
-screen_file_reload(struct mpdclient *c)
+screen_file_load_list(struct mpdclient *c, struct filelist *filelist)
 {
        struct mpd_connection *connection;
 
+       connection = mpdclient_get_connection(c);
+       if (connection == NULL)
+               return;
+
+       mpd_send_list_meta(connection, current_path);
+       filelist_recv(filelist, connection);
+
+       if (mpd_response_finish(connection))
+               filelist_sort_dir_play(filelist, compare_filelist_entry_path);
+       else
+               mpdclient_handle_error(c);
+}
+
+static void
+screen_file_reload(struct mpdclient *c)
+{
        if (browser.filelist != NULL)
                filelist_free(browser.filelist);
 
@@ -64,19 +80,7 @@ screen_file_reload(struct mpdclient *c)
                /* add a dummy entry for ./.. */
                filelist_append(browser.filelist, NULL);
 
-       if (!mpdclient_is_connected(c))
-               return;
-
-       connection = mpdclient_get_connection(c);
-
-       mpd_send_list_meta(connection, current_path);
-       filelist_recv(browser.filelist, connection);
-
-       if (mpd_response_finish(connection))
-               filelist_sort_dir_play(browser.filelist,
-                                      compare_filelist_entry_path);
-       else
-               mpdclient_handle_error(c);
+       screen_file_load_list(c, browser.filelist);
 
        list_window_set_length(browser.lw,
                               filelist_length(browser.filelist));
@@ -164,17 +168,16 @@ screen_file_handle_enter(struct mpdclient *c)
        return change_to_entry(c, entry);
 }
 
-static int
+static void
 handle_save(struct mpdclient *c)
 {
        struct list_window_range range;
        const char *defaultname = NULL;
        char *defaultname_utf8 = NULL;
-       int ret;
 
        list_window_get_range(browser.lw, &range);
        if (range.start == range.end)
-               return -1;
+               return;
 
        for (unsigned i = range.start; i < range.end; ++i) {
                struct filelist_entry *entry =
@@ -191,13 +194,11 @@ handle_save(struct mpdclient *c)
 
        if(defaultname)
                defaultname_utf8 = utf8_to_locale(defaultname);
-       ret = playlist_save(c, NULL, defaultname_utf8);
+       playlist_save(c, NULL, defaultname_utf8);
        g_free(defaultname_utf8);
-
-       return ret;
 }
 
-static int
+static void
 handle_delete(struct mpdclient *c)
 {
        struct mpd_connection *connection = mpdclient_get_connection(c);
@@ -207,6 +208,9 @@ handle_delete(struct mpdclient *c)
        char *str, *buf;
        int key;
 
+       if (connection == NULL)
+               return;
+
        list_window_get_range(browser.lw, &range);
        for (unsigned i = range.start; i < range.end; ++i) {
                struct filelist_entry *entry =
@@ -234,7 +238,7 @@ handle_delete(struct mpdclient *c)
                if( key != YES[0] ) {
                        /* translators: a dialog was aborted by the user */
                        screen_status_printf(_("Aborted"));
-                       return 0;
+                       return;
                }
 
                if (!mpd_run_rm(connection, mpd_playlist_get_path(playlist))) {
@@ -248,7 +252,6 @@ handle_delete(struct mpdclient *c)
                   user */
                screen_status_printf(_("Playlist deleted"));
        }
-       return 0;
 }
 
 static void