]> git.sesse.net Git - vlc/blobdiff - modules/gui/ncurses.c
vlc_plugin: fix non-LGPL plugins meta infos
[vlc] / modules / gui / ncurses.c
index cd85170395d6d265cce1abd1e3eb4a9c6c766c74..6de779b5f4690878ddd3e3beba837a14df7cd5de 100644 (file)
@@ -1,7 +1,7 @@
 /*****************************************************************************
  * ncurses.c : NCurses interface for vlc
  *****************************************************************************
- * Copyright © 2001-2010 the VideoLAN team
+ * Copyright © 2001-2011 the VideoLAN team
  * $Id$
  *
  * Authors: Sam Hocevar <sam@zoy.org>
 # include "config.h"
 #endif
 
-#include <vlc_common.h>
-#include <vlc_plugin.h>
-
 #define _XOPEN_SOURCE_EXTENDED 1
+
+#include <assert.h>
 #include <wchar.h>
+#include <sys/stat.h>
+#include <math.h>
+#include <errno.h>
+
+#define VLC_MODULE_LICENSE VLC_LICENSE_GPL_2_PLUS
+#include <vlc_common.h>
+#include <vlc_plugin.h>
 
 #include <ncurses.h>
 
 #include <vlc_interface.h>
 #include <vlc_vout.h>
-#include <vlc_aout.h>
 #include <vlc_charset.h>
 #include <vlc_input.h>
 #include <vlc_es.h>
 #include <vlc_playlist.h>
 #include <vlc_meta.h>
 #include <vlc_fs.h>
-
-#include <assert.h>
-
-#ifdef HAVE_SYS_STAT_H
-#   include <sys/stat.h>
-#endif
+#include <vlc_url.h>
 
 /*****************************************************************************
  * Local prototypes.
@@ -84,6 +84,8 @@ vlc_module_begin ()
     add_directory("browse-dir", NULL, BROWSE_TEXT, BROWSE_LONGTEXT, false)
 vlc_module_end ()
 
+#include "eject.c"
+
 /*****************************************************************************
  * intf_sys_t: description and status of ncurses interface
  *****************************************************************************/
@@ -102,7 +104,7 @@ enum
     BOX_STATS
 };
 
-static const char *box_title[] = {
+static const char box_title[][19] = {
     [BOX_NONE]      = "",
     [BOX_HELP]      = " Help ",
     [BOX_INFO]      = " Information ",
@@ -166,88 +168,92 @@ static const struct { short f; short b; } color_pairs[] =
 
 struct dir_entry_t
 {
-    bool        b_file;
-    char        *psz_path;
+    bool        file;
+    char        *path;
 };
 
 struct pl_item_t
 {
-    playlist_item_t *p_item;
-    char            *psz_display;
+    playlist_item_t *item;
+    char            *display;
 };
 
 struct intf_sys_t
 {
+    vlc_thread_t    thread;
     input_thread_t *p_input;
 
-    bool            b_color;
+    bool            color;
+    bool            exit;
+
+    /* rgb values for the color yellow */
+    short           yellow_r;
+    short           yellow_g;
+    short           yellow_b;
 
-    int             i_box_type;
-    int             i_box_y;            // start of box content
-    int             i_box_height;
-    int             i_box_lines_total;  // number of lines in the box
-    int             i_box_start;        // first line of box displayed
-    int             i_box_idx;          // selected line
+    int             box_type;
+    int             box_y;            // start of box content
+    int             box_height;
+    int             box_lines_total;  // number of lines in the box
+    int             box_start;        // first line of box displayed
+    int             box_idx;          // selected line
 
-    msg_subscription_t  *p_sub;         // message bank subscription
-    msg_item_t          *msgs[50];      // ring buffer
+    struct
+    {
+        int              type;
+        vlc_log_t       *item;
+        char            *msg;
+    } msgs[50];      // ring buffer
     int                 i_msgs;
+    int                 verbosity;
     vlc_mutex_t         msg_lock;
 
     /* Search Box context */
-    char            psz_search_chain[20];
-    char            *psz_old_search;
-    int             i_before_search;
+    char            search_chain[20];
 
     /* Open Box Context */
-    char            psz_open_chain[50];
+    char            open_chain[50];
 
     /* File Browser context */
-    char            *psz_current_dir;
-    int             i_dir_entries;
-    struct dir_entry_t  **pp_dir_entries;
-    bool            b_show_hidden_files;
+    char            *current_dir;
+    int             n_dir_entries;
+    struct dir_entry_t  **dir_entries;
+    bool            show_hidden_files;
 
     /* Playlist context */
-    struct pl_item_t    **pp_plist;
-    int             i_plist_entries;
-    bool            b_need_update;
-    bool            b_plidx_follow;
-    playlist_item_t *p_node;        /* current node */
+    struct pl_item_t    **plist;
+    int             plist_entries;
+    bool            need_update;
+    vlc_mutex_t     pl_lock;
+    bool            plidx_follow;
+    playlist_item_t *node;        /* current node */
 
 };
 
-struct msg_cb_data_t
-{
-    intf_sys_t *p_sys;
-};
-
 /*****************************************************************************
  * Directories
  *****************************************************************************/
 
-static void DirsDestroy(intf_sys_t *p_sys)
+static void DirsDestroy(intf_sys_t *sys)
 {
-    while (p_sys->i_dir_entries)
-    {
-        struct dir_entry_t *p_dir_entry;
-        p_dir_entry = p_sys->pp_dir_entries[--p_sys->i_dir_entries];
-        free(p_dir_entry->psz_path);
-        free(p_dir_entry);
+    while (sys->n_dir_entries) {
+        struct dir_entry_t *dir_entry = sys->dir_entries[--sys->n_dir_entries];
+        free(dir_entry->path);
+        free(dir_entry);
     }
-    free(p_sys->pp_dir_entries);
-    p_sys->pp_dir_entries = NULL;
+    free(sys->dir_entries);
+    sys->dir_entries = NULL;
 }
 
-static int comp_dir_entries(const void *pp_dir_entry1, const void *pp_dir_entry2)
+static int comdir_entries(const void *a, const void *b)
 {
-    struct dir_entry_t *p_dir_entry1 = *(struct dir_entry_t**)pp_dir_entry1;
-    struct dir_entry_t *p_dir_entry2 = *(struct dir_entry_t**)pp_dir_entry2;
+    struct dir_entry_t *dir_entry1 = *(struct dir_entry_t**)a;
+    struct dir_entry_t *dir_entry2 = *(struct dir_entry_t**)b;
 
-    if (p_dir_entry1->b_file == p_dir_entry2->b_file)
-        return strcasecmp(p_dir_entry1->psz_path, p_dir_entry2->psz_path);
+    if (dir_entry1->file == dir_entry2->file)
+        return strcasecmp(dir_entry1->path, dir_entry2->path);
 
-    return p_dir_entry1->b_file ? 1 : -1;
+    return dir_entry1->file ? 1 : -1;
 }
 
 static bool IsFile(const char *current_dir, const char *entry)
@@ -255,10 +261,8 @@ static bool IsFile(const char *current_dir, const char *entry)
     bool ret = true;
 #ifdef S_ISDIR
     char *uri;
-    struct stat st;
-
-    if (asprintf(&uri, "%s/%s", current_dir, entry) != -1)
-    {
+    if (asprintf(&uri, "%s" DIR_SEP "%s", current_dir, entry) != -1) {
+        struct stat st;
         ret = vlc_stat(uri, &st) || !S_ISDIR(st.st_mode);
         free(uri);
     }
@@ -266,321 +270,308 @@ static bool IsFile(const char *current_dir, const char *entry)
     return ret;
 }
 
-static void ReadDir(intf_thread_t *p_intf)
+static void ReadDir(intf_thread_t *intf)
 {
-    intf_sys_t *p_sys = p_intf->p_sys;
-    DIR *p_current_dir;
+    intf_sys_t *sys = intf->p_sys;
 
-    if (!p_sys->psz_current_dir || !*p_sys->psz_current_dir)
-    {
-        msg_Dbg(p_intf, "no current dir set");
+    if (!sys->current_dir || !*sys->current_dir) {
+        msg_Dbg(intf, "no current dir set");
         return;
     }
 
-    char *psz_entry;
-
-    /* Open the dir */
-    p_current_dir = vlc_opendir(p_sys->psz_current_dir);
-
-    if (!p_current_dir)
-    {
-        /* something went bad, get out of here ! */
-        msg_Warn(p_intf, "cannot open directory `%s' (%m)",
-                  p_sys->psz_current_dir);
+    DIR *current_dir = vlc_opendir(sys->current_dir);
+    if (!current_dir) {
+        msg_Warn(intf, "cannot open directory `%s' (%s)", sys->current_dir,
+                 vlc_strerror_c(errno));
         return;
     }
 
-    /* Clean the old shit */
-    DirsDestroy(p_sys);
-
-    /* while we still have entries in the directory */
-    while ((psz_entry = vlc_readdir(p_current_dir)))
-    {
-        struct dir_entry_t *p_dir_entry;
+    DirsDestroy(sys);
 
-        if (!p_sys->b_show_hidden_files)
-            if (*psz_entry == '.' && strcmp(psz_entry, ".."))
-                goto next;
+    const char *entry;
+    while ((entry = vlc_readdir(current_dir))) {
+        if (!sys->show_hidden_files && *entry == '.' && strcmp(entry, ".."))
+            continue;
 
-        if (!(p_dir_entry = malloc(sizeof *p_dir_entry)))
-            goto next;
+        struct dir_entry_t *dir_entry = malloc(sizeof *dir_entry);
+        if (unlikely(dir_entry == NULL))
+            continue;
 
-        p_dir_entry->b_file = IsFile(p_sys->psz_current_dir, psz_entry);
-        p_dir_entry->psz_path = psz_entry;
-        INSERT_ELEM(p_sys->pp_dir_entries, p_sys->i_dir_entries,
-             p_sys->i_dir_entries, p_dir_entry);
+        dir_entry->file = IsFile(sys->current_dir, entry);
+        dir_entry->path = xstrdup(entry);
+        INSERT_ELEM(sys->dir_entries, sys->n_dir_entries,
+             sys->n_dir_entries, dir_entry);
         continue;
-
-next:
-        free(psz_entry);
     }
 
-    /* Sort */
-    qsort(p_sys->pp_dir_entries, p_sys->i_dir_entries,
-           sizeof(struct dir_entry_t*), &comp_dir_entries);
+    qsort(sys->dir_entries, sys->n_dir_entries,
+           sizeof(struct dir_entry_t*), &comdir_entries);
 
-    closedir(p_current_dir);
+    closedir(current_dir);
 }
 
 /*****************************************************************************
  * Adjust index position after a change (list navigation or item switching)
  *****************************************************************************/
-static void CheckIdx(intf_sys_t *p_sys)
+static void CheckIdx(intf_sys_t *sys)
 {
-    int lines = p_sys->i_box_lines_total - 1;
-    int height = LINES - p_sys->i_box_y - 2;
+    int lines = sys->box_lines_total;
+    int height = LINES - sys->box_y - 2;
     if (height > lines - 1)
         height = lines - 1;
 
     /* make sure the new index is within the box */
-    if (p_sys->i_box_idx <= 0)
-    {
-        p_sys->i_box_idx = 0;
-        p_sys->i_box_start = 0;
-    }
-    else if (p_sys->i_box_idx >= lines - 1 && lines > 0)
-    {
-        p_sys->i_box_idx = lines - 1;
-        p_sys->i_box_start = p_sys->i_box_idx - height;
+    if (sys->box_idx <= 0) {
+        sys->box_idx = 0;
+        sys->box_start = 0;
+    } else if (sys->box_idx >= lines - 1 && lines > 0) {
+        sys->box_idx = lines - 1;
+        sys->box_start = sys->box_idx - height;
     }
 
     /* Fix box start (1st line of the box displayed) */
-    if (p_sys->i_box_idx < p_sys->i_box_start || 
-        p_sys->i_box_idx > height + p_sys->i_box_start + 1)
-    {
-        p_sys->i_box_start = p_sys->i_box_idx - height/2;
-        if (p_sys->i_box_start < 0)
-            p_sys->i_box_start = 0;
+    if (sys->box_idx < sys->box_start ||
+        sys->box_idx > height + sys->box_start + 1) {
+        sys->box_start = sys->box_idx - height/2;
+        if (sys->box_start < 0)
+            sys->box_start = 0;
+    } else if (sys->box_idx == sys->box_start - 1) {
+        sys->box_start--;
+    } else if (sys->box_idx == height + sys->box_start + 1) {
+        sys->box_start++;
     }
-    else if (p_sys->i_box_idx == p_sys->i_box_start - 1)
-        p_sys->i_box_start--;
-    else if (p_sys->i_box_idx == height + p_sys->i_box_start + 1)
-        p_sys->i_box_start++;
 }
 
 /*****************************************************************************
  * Playlist
  *****************************************************************************/
-static void PlaylistDestroy(intf_sys_t *p_sys)
+static void PlaylistDestroy(intf_sys_t *sys)
 {
-    while (p_sys->i_plist_entries)
-    {
-        struct pl_item_t *p_pl_item = p_sys->pp_plist[--p_sys->i_plist_entries];
-        free(p_pl_item->psz_display);
+    while (sys->plist_entries) {
+        struct pl_item_t *p_pl_item = sys->plist[--sys->plist_entries];
+        free(p_pl_item->display);
         free(p_pl_item);
     }
-    free(p_sys->pp_plist);
-    p_sys->pp_plist = NULL;
+    free(sys->plist);
+    sys->plist = NULL;
 }
 
-static bool PlaylistAddChild(intf_sys_t *p_sys, playlist_item_t *p_child,
+static bool PlaylistAddChild(intf_sys_t *sys, playlist_item_t *p_child,
                              const char *c, const char d)
 {
     int ret;
-    char *psz_name = input_item_GetTitleFbName(p_child->p_input);
+    char *name = input_item_GetTitleFbName(p_child->p_input);
     struct pl_item_t *p_pl_item = malloc(sizeof *p_pl_item);
 
-    if (!psz_name || !p_pl_item)
+    if (!name || !p_pl_item)
         goto error;
 
-    p_pl_item->p_item = p_child;
+    p_pl_item->item = p_child;
 
     if (c && *c)
-        ret = asprintf(&p_pl_item->psz_display, "%s%c-%s", c, d, psz_name);
+        ret = asprintf(&p_pl_item->display, "%s%c-%s", c, d, name);
     else
-        ret = asprintf(&p_pl_item->psz_display, " %s", psz_name);
+        ret = asprintf(&p_pl_item->display, " %s", name);
 
-    free(psz_name);
-    psz_name = NULL;
+    free(name);
+    name = NULL;
 
     if (ret == -1)
         goto error;
 
-    INSERT_ELEM(p_sys->pp_plist, p_sys->i_plist_entries,
-                 p_sys->i_plist_entries, p_pl_item);
+    INSERT_ELEM(sys->plist, sys->plist_entries,
+                 sys->plist_entries, p_pl_item);
 
     return true;
 
 error:
-    free(psz_name);
+    free(name);
     free(p_pl_item);
     return false;
 }
 
-static void PlaylistAddNode(intf_sys_t *p_sys, playlist_item_t *p_node,
+static void PlaylistAddNode(intf_sys_t *sys, playlist_item_t *node,
                             const char *c)
 {
-    for(int k = 0; k < p_node->i_children; k++)
-    {
-        playlist_item_t *p_child = p_node->pp_children[k];
-        char d = k == p_node->i_children - 1 ? '`' : '|';
-        if(!PlaylistAddChild(p_sys, p_child, c, d))
+    for (int k = 0; k < node->i_children; k++) {
+        bool last = k == node->i_children - 1;
+        playlist_item_t *p_child = node->pp_children[k];
+        if (!PlaylistAddChild(sys, p_child, c, last ? '`' : '|'))
             return;
 
         if (p_child->i_children <= 0)
             continue;
 
-        if (*c)
-        {
-            char *psz_tmp;
-            if (asprintf(&psz_tmp, "%s%c ", c,
-                     k == p_node->i_children - 1 ? ' ' : '|') == -1)
+        if (*c) {
+            char *tmp;
+            if (asprintf(&tmp, "%s%c ", c, last ? ' ' : '|') == -1)
                 return;
-            PlaylistAddNode(p_sys, p_child, psz_tmp);
-            free(psz_tmp);
+            PlaylistAddNode(sys, p_child, tmp);
+            free(tmp);
+        } else {
+            PlaylistAddNode(sys, p_child, " ");
         }
-        else
-            PlaylistAddNode(p_sys, p_child, " ");
     }
 }
 
-static void PlaylistRebuild(intf_thread_t *p_intf)
+static void PlaylistRebuild(intf_thread_t *intf)
 {
-    intf_sys_t *p_sys = p_intf->p_sys;
-    playlist_t *p_playlist = pl_Get(p_intf);
+    intf_sys_t *sys = intf->p_sys;
+    playlist_t *p_playlist = pl_Get(intf);
 
-    PL_LOCK;
-    PlaylistDestroy(p_sys);
-    PlaylistAddNode(p_sys, p_playlist->p_root_onelevel, "");
-    PL_UNLOCK;
+    PlaylistDestroy(sys);
+    PlaylistAddNode(sys, p_playlist->p_root_onelevel, "");
 }
 
-static int PlaylistChanged(vlc_object_t *p_this, const char *psz_variable,
+static int ItemChanged(vlc_object_t *p_this, const char *variable,
                             vlc_value_t oval, vlc_value_t nval, void *param)
 {
-    VLC_UNUSED(p_this); VLC_UNUSED(psz_variable);
+    VLC_UNUSED(p_this); VLC_UNUSED(variable);
     VLC_UNUSED(oval); VLC_UNUSED(nval);
 
-    intf_thread_t *p_intf   = (intf_thread_t *)param;
-    intf_sys_t *p_sys       = p_intf->p_sys;
-    playlist_item_t *p_node = playlist_CurrentPlayingItem(pl_Get(p_intf));
+    intf_sys_t *sys = ((intf_thread_t *)param)->p_sys;
 
-    p_sys->b_need_update = true;
-    p_sys->p_node = p_node ? p_node->p_parent : NULL;
+    vlc_mutex_lock(&sys->pl_lock);
+    sys->need_update = true;
+    vlc_mutex_unlock(&sys->pl_lock);
 
     return VLC_SUCCESS;
 }
 
-/* Playlist suxx */
-/* This function have to be called with the playlist locked */
-static inline bool PlaylistIsPlaying(playlist_t *p_playlist,
-                                     playlist_item_t *p_item)
+static int PlaylistChanged(vlc_object_t *p_this, const char *variable,
+                            vlc_value_t oval, vlc_value_t nval, void *param)
 {
-    playlist_item_t *p_played_item = playlist_CurrentPlayingItem(p_playlist);
-    return p_item                && p_played_item
-        && p_item->p_input       && p_played_item->p_input
-        && p_item->p_input->i_id == p_played_item->p_input->i_id;
+    VLC_UNUSED(p_this); VLC_UNUSED(variable);
+    VLC_UNUSED(oval); VLC_UNUSED(nval);
+    intf_thread_t *intf   = (intf_thread_t *)param;
+    intf_sys_t *sys       = intf->p_sys;
+    playlist_item_t *node = playlist_CurrentPlayingItem(pl_Get(intf));
+
+    vlc_mutex_lock(&sys->pl_lock);
+    sys->need_update = true;
+    sys->node = node ? node->p_parent : NULL;
+    vlc_mutex_unlock(&sys->pl_lock);
+
+    return VLC_SUCCESS;
 }
 
-static int SubSearchPlaylist(intf_sys_t *p_sys, char *psz_searchstring,
+/* Playlist suxx */
+static int SubSearchPlaylist(intf_sys_t *sys, char *searchstring,
                               int i_start, int i_stop)
 {
-    for(int i = i_start + 1; i < i_stop; i++)
-        if (strcasestr(p_sys->pp_plist[i]->psz_display, psz_searchstring))
+    for (int i = i_start + 1; i < i_stop; i++)
+        if (strcasestr(sys->plist[i]->display, searchstring))
             return i;
 
     return -1;
 }
 
-static void SearchPlaylist(intf_sys_t *p_sys, char *psz_searchstring)
+static void SearchPlaylist(intf_sys_t *sys)
 {
-    int i_item, i_first = p_sys->i_before_search;
-
+    char *str = sys->search_chain;
+    int i_first = sys->box_idx;
     if (i_first < 0)
         i_first = 0;
 
-    if (!psz_searchstring || !*psz_searchstring)
-    {
-        p_sys->i_box_idx = p_sys->i_before_search;
+    if (!str || !*str)
         return;
-    }
 
-    i_item = SubSearchPlaylist(p_sys, psz_searchstring, i_first + 1,
-                               p_sys->i_plist_entries);
+    int i_item = SubSearchPlaylist(sys, str, i_first + 1, sys->plist_entries);
     if (i_item < 0)
-        i_item = SubSearchPlaylist(p_sys, psz_searchstring, 0, i_first);
+        i_item = SubSearchPlaylist(sys, str, 0, i_first);
 
-    if (i_item > 0)
-    {
-        p_sys->i_box_idx = i_item;
-        CheckIdx(p_sys);
+    if (i_item > 0) {
+        sys->box_idx = i_item;
+        CheckIdx(sys);
     }
 }
 
-static inline bool IsIndex(intf_sys_t *p_sys, playlist_t *p_playlist, int i)
+static inline bool IsIndex(intf_sys_t *sys, playlist_t *p_playlist, int i)
 {
-    playlist_item_t *p_item = p_sys->pp_plist[i]->p_item;
-    return (p_item->i_children == 0 && p_item == p_sys->p_node) ||
-            PlaylistIsPlaying(p_playlist, p_item);
+    playlist_item_t *item = sys->plist[i]->item;
+
+    PL_ASSERT_LOCKED;
+
+    vlc_mutex_lock(&sys->pl_lock);
+    if (item->i_children == 0 && item == sys->node) {
+        vlc_mutex_unlock(&sys->pl_lock);
+        return true;
+    }
+    vlc_mutex_unlock(&sys->pl_lock);
+
+    playlist_item_t *p_played_item = playlist_CurrentPlayingItem(p_playlist);
+    if (p_played_item && item->p_input && p_played_item->p_input)
+        return item->p_input->i_id == p_played_item->p_input->i_id;
+
+    return false;
 }
 
-static void FindIndex(intf_sys_t *p_sys, playlist_t *p_playlist)
+static void FindIndex(intf_sys_t *sys, playlist_t *p_playlist)
 {
-    int plidx = p_sys->i_box_idx;
-    int max = p_sys->i_plist_entries;
+    int plidx = sys->box_idx;
+    int max = sys->plist_entries;
 
     PL_LOCK;
 
-    if (!IsIndex(p_sys, p_playlist, plidx))
-        for(int i = 0; i < max; i++)
-            if (IsIndex(p_sys, p_playlist, i))
-            {
-                p_sys->i_box_idx = i;
-                CheckIdx(p_sys);
+    if (!IsIndex(sys, p_playlist, plidx))
+        for (int i = 0; i < max; i++)
+            if (IsIndex(sys, p_playlist, i)) {
+                sys->box_idx = i;
+                CheckIdx(sys);
                 break;
             }
 
     PL_UNLOCK;
 
-    p_sys->b_plidx_follow = true;
+    sys->plidx_follow = true;
 }
 
 /****************************************************************************
  * Drawing
  ****************************************************************************/
 
-static void start_color_and_pairs(intf_thread_t *p_intf)
+static void start_color_and_pairs(intf_thread_t *intf)
 {
-    if (!has_colors())
-    {
-        p_intf->p_sys->b_color = false;
-        msg_Warn(p_intf, "Terminal doesn't support colors");
+    intf_sys_t *sys = intf->p_sys;
+
+    if (!has_colors()) {
+        sys->color = false;
+        msg_Warn(intf, "Terminal doesn't support colors");
         return;
     }
 
     start_color();
-    for(int i = C_DEFAULT + 1; i < C_MAX; i++)
+    for (int i = C_DEFAULT + 1; i < C_MAX; i++)
         init_pair(i, color_pairs[i].f, color_pairs[i].b);
 
     /* untested, in all my terminals, !can_change_color() --funman */
-    if (can_change_color())
+    if (can_change_color()) {
+        color_content(COLOR_YELLOW, &sys->yellow_r, &sys->yellow_g, &sys->yellow_b);
         init_color(COLOR_YELLOW, 960, 500, 0); /* YELLOW -> ORANGE */
+    }
 }
 
-static void DrawBox(int y, int h, bool b_color, const char *title)
+static void DrawBox(int y, int h, bool color, const char *title)
 {
-    int i_len;
     int w = COLS;
-
     if (w <= 3 || h <= 0)
         return;
 
-    if (b_color) color_set(C_BOX, NULL);
+    if (color) color_set(C_BOX, NULL);
 
     if (!title) title = "";
-    i_len = strlen(title);
+    int len = strlen(title);
 
-    if (i_len > w - 2)
-        i_len = w - 2;
+    if (len > w - 2)
+        len = w - 2;
 
     mvaddch(y, 0,    ACS_ULCORNER);
-    mvhline(y, 1,  ACS_HLINE, (w-i_len-2)/2);
-    mvprintw(y, 1+(w-i_len-2)/2, "%s", title);
-    mvhline(y, (w-i_len)/2+i_len,  ACS_HLINE, w - 1 - ((w-i_len)/2+i_len));
+    mvhline(y, 1,  ACS_HLINE, (w-len-2)/2);
+    mvprintw(y, 1+(w-len-2)/2, "%s", title);
+    mvhline(y, (w-len)/2+len,  ACS_HLINE, w - 1 - ((w-len)/2+len));
     mvaddch(y, w-1,ACS_URCORNER);
 
-    for(int i = 0; i < h; i++)
-    {
+    for (int i = 0; i < h; i++) {
         mvaddch(++y, 0,   ACS_VLINE);
         mvaddch(y, w-1, ACS_VLINE);
     }
@@ -588,7 +579,7 @@ static void DrawBox(int y, int h, bool b_color, const char *title)
     mvaddch(++y, 0,   ACS_LLCORNER);
     mvhline(y,   1,   ACS_HLINE, w - 2);
     mvaddch(y,   w-1, ACS_LRCORNER);
-    if (b_color) color_set(C_DEFAULT, NULL);
+    if (color) color_set(C_DEFAULT, NULL);
 }
 
 static void DrawEmptyLine(int y, int x, int w)
@@ -611,46 +602,44 @@ static void mvnprintw(int y, int x, int w, const char *p_fmt, ...)
 {
     va_list  vl_args;
     char    *p_buf;
-    int      i_len;
+    int      len;
 
     if (w <= 0)
         return;
 
     va_start(vl_args, p_fmt);
-    if (vasprintf(&p_buf, p_fmt, vl_args) == -1)
-        return;
+    int i_ret = vasprintf(&p_buf, p_fmt, vl_args);
     va_end(vl_args);
 
-    i_len = strlen(p_buf);
+    if (i_ret == -1)
+        return;
+
+    len = strlen(p_buf);
 
-    wchar_t psz_wide[i_len + 1];
+    wchar_t wide[len + 1];
 
     EnsureUTF8(p_buf);
-    size_t i_char_len = mbstowcs(psz_wide, p_buf, i_len);
+    size_t i_char_len = mbstowcs(wide, p_buf, len);
 
     size_t i_width; /* number of columns */
 
-    if (i_char_len == (size_t)-1) /* an invalid character was encountered */
-    {
+    if (i_char_len == (size_t)-1) /* an invalid character was encountered */ {
         free(p_buf);
         return;
     }
 
-    i_width = wcswidth(psz_wide, i_char_len);
-    if (i_width == (size_t)-1)
-    {
+    i_width = wcswidth(wide, i_char_len);
+    if (i_width == (size_t)-1) {
         /* a non printable character was encountered */
         i_width = 0;
-        for(unsigned i = 0 ; i < i_char_len ; i++)
-        {
-            int i_cwidth = wcwidth(psz_wide[i]);
+        for (unsigned i = 0 ; i < i_char_len ; i++) {
+            int i_cwidth = wcwidth(wide[i]);
             if (i_cwidth != -1)
                 i_width += i_cwidth;
         }
     }
 
-    if (i_width <= (size_t)w)
-    {
+    if (i_width <= (size_t)w) {
         mvprintw(y, x, "%s", p_buf);
         mvhline(y, x + i_width, ' ', w - i_width);
         free(p_buf);
@@ -659,415 +648,399 @@ static void mvnprintw(int y, int x, int w, const char *p_fmt, ...)
 
     int i_total_width = 0;
     int i = 0;
-    while (i_total_width < w)
-    {
-        i_total_width += wcwidth(psz_wide[i]);
-        if (w > 7 && i_total_width >= w/2)
-        {
-            psz_wide[i  ] = '.';
-            psz_wide[i+1] = '.';
-            i_total_width -= wcwidth(psz_wide[i]) - 2;
-            if (i > 0)
-            {
+    while (i_total_width < w) {
+        i_total_width += wcwidth(wide[i]);
+        if (w > 7 && i_total_width >= w/2) {
+            wide[i  ] = '.';
+            wide[i+1] = '.';
+            i_total_width -= wcwidth(wide[i]) - 2;
+            if (i > 0) {
                 /* we require this check only if at least one character
                  * 4 or more columns wide exists (which i doubt) */
-                psz_wide[i-1] = '.';
-                i_total_width -= wcwidth(psz_wide[i-1]) - 1;
+                wide[i-1] = '.';
+                i_total_width -= wcwidth(wide[i-1]) - 1;
             }
 
             /* find the widest string */
             int j, i_2nd_width = 0;
-            for(j = i_char_len - 1; i_2nd_width < w - i_total_width; j--)
-                i_2nd_width += wcwidth(psz_wide[j]);
+            for (j = i_char_len - 1; i_2nd_width < w - i_total_width; j--)
+                i_2nd_width += wcwidth(wide[j]);
 
             /* we already have i_total_width columns filled, and we can't
              * have more than w columns */
             if (i_2nd_width > w - i_total_width)
                 j++;
 
-            wmemmove(&psz_wide[i+2], &psz_wide[j+1], i_char_len - j - 1);
-            psz_wide[i + 2 + i_char_len - j - 1] = '\0';
+            wmemmove(&wide[i+2], &wide[j+1], i_char_len - j - 1);
+            wide[i + 2 + i_char_len - j - 1] = '\0';
             break;
         }
         i++;
     }
     if (w <= 7) /* we don't add the '...' else we lose too much chars */
-        psz_wide[i] = '\0';
+        wide[i] = '\0';
 
-    size_t i_wlen = wcslen(psz_wide) * 6 + 1; /* worst case */
-    char psz_ellipsized[i_wlen];
-    wcstombs(psz_ellipsized, psz_wide, i_wlen);
-    mvprintw(y, x, "%s", psz_ellipsized);
+    size_t i_wlen = wcslen(wide) * 6 + 1; /* worst case */
+    char ellipsized[i_wlen];
+    wcstombs(ellipsized, wide, i_wlen);
+    mvprintw(y, x, "%s", ellipsized);
 
     free(p_buf);
 }
 
-static void MainBoxWrite(intf_sys_t *p_sys, int l, const char *p_fmt, ...)
+static void MainBoxWrite(intf_sys_t *sys, int l, const char *p_fmt, ...)
 {
     va_list     vl_args;
     char        *p_buf;
-    bool        b_selected = l == p_sys->i_box_idx;
+    bool        b_selected = l == sys->box_idx;
 
-    if (l < p_sys->i_box_start || l - p_sys->i_box_start >= p_sys->i_box_height)
+    if (l < sys->box_start || l - sys->box_start >= sys->box_height)
         return;
 
     va_start(vl_args, p_fmt);
-    if (vasprintf(&p_buf, p_fmt, vl_args) == -1)
-        return;
+    int i_ret = vasprintf(&p_buf, p_fmt, vl_args);
     va_end(vl_args);
+    if (i_ret == -1)
+        return;
 
     if (b_selected) attron(A_REVERSE);
-    mvnprintw(p_sys->i_box_y + l - p_sys->i_box_start, 1, COLS - 2, "%s", p_buf);
+    mvnprintw(sys->box_y + l - sys->box_start, 1, COLS - 2, "%s", p_buf);
     if (b_selected) attroff(A_REVERSE);
 
     free(p_buf);
 }
 
-static int SubDrawObject(intf_sys_t *p_sys, int l, vlc_object_t *p_obj, int i_level, const char *prefix)
+static int SubDrawObject(intf_sys_t *sys, int l, vlc_object_t *p_obj, int i_level, const char *prefix)
 {
-    int x = 2 * i_level++;
-    char *psz_name = vlc_object_get_name(p_obj);
-    vlc_list_t *list;
-    char space[x+3];
-    memset(space, ' ', x);
-    space[x] = '\0';
-
-    if (psz_name)
-    {
-        MainBoxWrite(p_sys, l++, "%s%s%s \"%s\" (%p)", space, prefix,
-                      p_obj->psz_object_type, psz_name, p_obj);
-        free(psz_name);
-    }
-    else
-        MainBoxWrite(p_sys, l++, "%s%s%s (%p)", space, prefix,
-                      p_obj->psz_object_type, p_obj);
-
-    list = vlc_list_children(p_obj);
-    for(int i = 0; i < list->i_count ; i++)
-    {
-        l = SubDrawObject(p_sys, l, list->p_values[i].p_object, i_level,
+    char *name = vlc_object_get_name(p_obj);
+    MainBoxWrite(sys, l++, "%*s%s%s \"%s\" (%p)", 2 * i_level++, "", prefix,
+                  p_obj->psz_object_type, name ? name : "", p_obj);
+    free(name);
+
+    vlc_list_t *list = vlc_list_children(p_obj);
+    for (int i = 0; i < list->i_count ; i++) {
+        l = SubDrawObject(sys, l, list->p_values[i].p_address, i_level,
             (i == list->i_count - 1) ? "`-" : "|-" );
     }
     vlc_list_release(list);
     return l;
 }
 
-static int DrawObjects(intf_thread_t *p_intf) 
+static int DrawObjects(intf_thread_t *intf)
 {
-    return SubDrawObject(p_intf->p_sys, 0, VLC_OBJECT(p_intf->p_libvlc), 0, "");
+    return SubDrawObject(intf->p_sys, 0, VLC_OBJECT(intf->p_libvlc), 0, "");
 }
 
-static int DrawMeta(intf_thread_t *p_intf)
+static int DrawMeta(intf_thread_t *intf)
 {
-    intf_sys_t *p_sys = p_intf->p_sys;
-    input_thread_t *p_input = p_sys->p_input;
-    input_item_t *p_item;
+    intf_sys_t *sys = intf->p_sys;
+    input_thread_t *p_input = sys->p_input;
+    input_item_t *item;
     int l = 0;
 
     if (!p_input)
         return 0;
 
-    p_item = input_GetItem(p_input);
-    vlc_mutex_lock(&p_item->lock);
-    for(int i=0; i<VLC_META_TYPE_COUNT; i++)
-    {
-        const char *psz_meta = vlc_meta_Get(p_item->p_meta, i);
-        if (!psz_meta || !*psz_meta)
+    item = input_GetItem(p_input);
+    vlc_mutex_lock(&item->lock);
+    for (int i=0; i<VLC_META_TYPE_COUNT; i++) {
+        const char *meta = vlc_meta_Get(item->p_meta, i);
+        if (!meta || !*meta)
             continue;
 
-        if (p_sys->b_color) color_set(C_CATEGORY, NULL);
-        MainBoxWrite(p_sys, l++, "  [%s]",
-                     vlc_meta_TypeToLocalizedString(i));
-        if (p_sys->b_color) color_set(C_DEFAULT, NULL);
-        MainBoxWrite(p_sys, l++, "      %s", psz_meta);
+        if (sys->color) color_set(C_CATEGORY, NULL);
+        MainBoxWrite(sys, l++, "  [%s]", vlc_meta_TypeToLocalizedString(i));
+        if (sys->color) color_set(C_DEFAULT, NULL);
+        MainBoxWrite(sys, l++, "      %s", meta);
     }
-    vlc_mutex_unlock(&p_item->lock);
+    vlc_mutex_unlock(&item->lock);
 
     return l;
 }
 
-static int DrawInfo(intf_thread_t *p_intf)
+static int DrawInfo(intf_thread_t *intf)
 {
-    intf_sys_t *p_sys = p_intf->p_sys;
-    input_thread_t *p_input = p_sys->p_input;
-    input_item_t *p_item;
+    intf_sys_t *sys = intf->p_sys;
+    input_thread_t *p_input = sys->p_input;
+    input_item_t *item;
     int l = 0;
 
     if (!p_input)
         return 0;
 
-    p_item = input_GetItem(p_input);
-    vlc_mutex_lock(&p_item->lock);
-    for(int i = 0; i < p_item->i_categories; i++)
-    {
-        info_category_t *p_category = p_item->pp_categories[i];
-        if (p_sys->b_color) color_set(C_CATEGORY, NULL);
-        MainBoxWrite(p_sys, l++, _("  [%s]"), p_category->psz_name);
-        if (p_sys->b_color) color_set(C_DEFAULT, NULL);
-        for(int j = 0; j < p_category->i_infos; j++)
-        {
+    item = input_GetItem(p_input);
+    vlc_mutex_lock(&item->lock);
+    for (int i = 0; i < item->i_categories; i++) {
+        info_category_t *p_category = item->pp_categories[i];
+        if (sys->color) color_set(C_CATEGORY, NULL);
+        MainBoxWrite(sys, l++, _("  [%s]"), p_category->psz_name);
+        if (sys->color) color_set(C_DEFAULT, NULL);
+        for (int j = 0; j < p_category->i_infos; j++) {
             info_t *p_info = p_category->pp_infos[j];
-            MainBoxWrite(p_sys, l++, _("      %s: %s"),
+            MainBoxWrite(sys, l++, _("      %s: %s"),
                          p_info->psz_name, p_info->psz_value);
         }
     }
-    vlc_mutex_unlock(&p_item->lock);
+    vlc_mutex_unlock(&item->lock);
 
     return l;
 }
 
-static int DrawStats(intf_thread_t *p_intf)
+static int DrawStats(intf_thread_t *intf)
 {
-    intf_sys_t *p_sys = p_intf->p_sys;
-    input_thread_t *p_input = p_sys->p_input;
-    input_item_t *p_item;
+    intf_sys_t *sys = intf->p_sys;
+    input_thread_t *p_input = sys->p_input;
+    input_item_t *item;
     input_stats_t *p_stats;
     int l = 0, i_audio = 0, i_video = 0;
 
     if (!p_input)
         return 0;
 
-    p_item = input_GetItem(p_input);
-    assert(p_item);
+    item = input_GetItem(p_input);
+    assert(item);
 
-    vlc_mutex_lock(&p_item->lock);
-    p_stats = p_item->p_stats;
+    vlc_mutex_lock(&item->lock);
+    p_stats = item->p_stats;
     vlc_mutex_lock(&p_stats->lock);
 
-    for(int i = 0; i < p_item->i_es ; i++)
-    {
-        i_audio += (p_item->es[i]->i_cat == AUDIO_ES);
-        i_video += (p_item->es[i]->i_cat == VIDEO_ES);
+    for (int i = 0; i < item->i_es ; i++) {
+        i_audio += (item->es[i]->i_cat == AUDIO_ES);
+        i_video += (item->es[i]->i_cat == VIDEO_ES);
     }
 
     /* Input */
-    if (p_sys->b_color) color_set(C_CATEGORY, NULL);
-    MainBoxWrite(p_sys, l++, _("  [Incoming]"));
-    if (p_sys->b_color) color_set(C_DEFAULT, NULL);
-    MainBoxWrite(p_sys, l++, _("      input bytes read : %8.0f KiB"),
+    if (sys->color) color_set(C_CATEGORY, NULL);
+    MainBoxWrite(sys, l++, _("+-[Incoming]"));
+    if (sys->color) color_set(C_DEFAULT, NULL);
+    MainBoxWrite(sys, l++, _("| input bytes read : %8.0f KiB"),
             (float)(p_stats->i_read_bytes)/1024);
-    MainBoxWrite(p_sys, l++, _("      input bitrate    :   %6.0f kb/s"),
+    MainBoxWrite(sys, l++, _("| input bitrate    :   %6.0f kb/s"),
             p_stats->f_input_bitrate*8000);
-    MainBoxWrite(p_sys, l++, _("      demux bytes read : %8.0f KiB"),
+    MainBoxWrite(sys, l++, _("| demux bytes read : %8.0f KiB"),
             (float)(p_stats->i_demux_read_bytes)/1024);
-    MainBoxWrite(p_sys, l++, _("      demux bitrate    :   %6.0f kb/s"),
+    MainBoxWrite(sys, l++, _("| demux bitrate    :   %6.0f kb/s"),
             p_stats->f_demux_bitrate*8000);
 
     /* Video */
-    if (i_video)
-    {
-        if (p_sys->b_color) color_set(C_CATEGORY, NULL);
-        MainBoxWrite(p_sys, l++, _("  [Video Decoding]"));
-        if (p_sys->b_color) color_set(C_DEFAULT, NULL);
-        MainBoxWrite(p_sys, l++, _("      video decoded    :    %"PRId64),
+    if (i_video) {
+        if (sys->color) color_set(C_CATEGORY, NULL);
+        MainBoxWrite(sys, l++, _("+-[Video Decoding]"));
+        if (sys->color) color_set(C_DEFAULT, NULL);
+        MainBoxWrite(sys, l++, _("| video decoded    :    %5"PRIi64),
                 p_stats->i_decoded_video);
-        MainBoxWrite(p_sys, l++, _("      frames displayed :    %"PRId64),
+        MainBoxWrite(sys, l++, _("| frames displayed :    %5"PRIi64),
                 p_stats->i_displayed_pictures);
-        MainBoxWrite(p_sys, l++, _("      frames lost      :    %"PRId64),
+        MainBoxWrite(sys, l++, _("| frames lost      :    %5"PRIi64),
                 p_stats->i_lost_pictures);
     }
     /* Audio*/
-    if (i_audio)
-    {
-        if (p_sys->b_color) color_set(C_CATEGORY, NULL);
-        MainBoxWrite(p_sys, l++, _("  [Audio Decoding]"));
-        if (p_sys->b_color) color_set(C_DEFAULT, NULL);
-        MainBoxWrite(p_sys, l++, _("      audio decoded    :    %"PRId64),
+    if (i_audio) {
+        if (sys->color) color_set(C_CATEGORY, NULL);
+        MainBoxWrite(sys, l++, _("+-[Audio Decoding]"));
+        if (sys->color) color_set(C_DEFAULT, NULL);
+        MainBoxWrite(sys, l++, _("| audio decoded    :    %5"PRIi64),
                 p_stats->i_decoded_audio);
-        MainBoxWrite(p_sys, l++, _("      buffers played   :    %"PRId64),
+        MainBoxWrite(sys, l++, _("| buffers played   :    %5"PRIi64),
                 p_stats->i_played_abuffers);
-        MainBoxWrite(p_sys, l++, _("      buffers lost     :    %"PRId64),
+        MainBoxWrite(sys, l++, _("| buffers lost     :    %5"PRIi64),
                 p_stats->i_lost_abuffers);
     }
     /* Sout */
-    if (p_sys->b_color) color_set(C_CATEGORY, NULL);
-    MainBoxWrite(p_sys, l++, _("  [Streaming]"));
-    if (p_sys->b_color) color_set(C_DEFAULT, NULL);
-    MainBoxWrite(p_sys, l++, _("      packets sent     :    %5i"), p_stats->i_sent_packets);
-    MainBoxWrite(p_sys, l++, _("      bytes sent       : %8.0f KiB"),
+    if (sys->color) color_set(C_CATEGORY, NULL);
+    MainBoxWrite(sys, l++, _("+-[Streaming]"));
+    if (sys->color) color_set(C_DEFAULT, NULL);
+    MainBoxWrite(sys, l++, _("| packets sent     :    %5"PRIi64), p_stats->i_sent_packets);
+    MainBoxWrite(sys, l++, _("| bytes sent       : %8.0f KiB"),
             (float)(p_stats->i_sent_bytes)/1025);
-    MainBoxWrite(p_sys, l++, _("      sending bitrate  :   %6.0f kb/s"),
+    MainBoxWrite(sys, l++, _("| sending bitrate  :   %6.0f kb/s"),
             p_stats->f_send_bitrate*8000);
-    if (p_sys->b_color) color_set(C_DEFAULT, NULL);
+    if (sys->color) color_set(C_DEFAULT, NULL);
 
     vlc_mutex_unlock(&p_stats->lock);
-    vlc_mutex_unlock(&p_item->lock);
+    vlc_mutex_unlock(&item->lock);
 
     return l;
 }
 
-static int DrawHelp(intf_thread_t *p_intf)
+static int DrawHelp(intf_thread_t *intf)
 {
-    intf_sys_t *p_sys = p_intf->p_sys;
+    intf_sys_t *sys = intf->p_sys;
     int l = 0;
 
-    if (p_sys->b_color) color_set(C_CATEGORY, NULL);
-    MainBoxWrite(p_sys, l++, _("[Display]"));
-    if (p_sys->b_color) color_set(C_DEFAULT, NULL);
-    MainBoxWrite(p_sys, l++, _("     h,H         Show/Hide help box"));
-    MainBoxWrite(p_sys, l++, _("     i           Show/Hide info box"));
-    MainBoxWrite(p_sys, l++, _("     m           Show/Hide metadata box"));
-    MainBoxWrite(p_sys, l++, _("     L           Show/Hide messages box"));
-    MainBoxWrite(p_sys, l++, _("     P           Show/Hide playlist box"));
-    MainBoxWrite(p_sys, l++, _("     B           Show/Hide filebrowser"));
-    MainBoxWrite(p_sys, l++, _("     x           Show/Hide objects box"));
-    MainBoxWrite(p_sys, l++, _("     S           Show/Hide statistics box"));
-    MainBoxWrite(p_sys, l++, _("     Esc         Close Add/Search entry"));
-    MainBoxWrite(p_sys, l++, _("     Ctrl-l      Refresh the screen"));
-    MainBoxWrite(p_sys, l++, "");
-
-    if (p_sys->b_color) color_set(C_CATEGORY, NULL);
-    MainBoxWrite(p_sys, l++, _("[Global]"));
-    if (p_sys->b_color) color_set(C_DEFAULT, NULL);
-    MainBoxWrite(p_sys, l++, _("     q, Q, Esc   Quit"));
-    MainBoxWrite(p_sys, l++, _("     s           Stop"));
-    MainBoxWrite(p_sys, l++, _("     <space>     Pause/Play"));
-    MainBoxWrite(p_sys, l++, _("     f           Toggle Fullscreen"));
-    MainBoxWrite(p_sys, l++, _("     n, p        Next/Previous playlist item"));
-    MainBoxWrite(p_sys, l++, _("     [, ]        Next/Previous title"));
-    MainBoxWrite(p_sys, l++, _("     <, >        Next/Previous chapter"));
-    MainBoxWrite(p_sys, l++, _("     ←, →        Seek -/+ 1%%"));
-    MainBoxWrite(p_sys, l++, _("     a, z        Volume Up/Down"));
-    MainBoxWrite(p_sys, l++, "");
-
-    if (p_sys->b_color) color_set(C_CATEGORY, NULL);
-    MainBoxWrite(p_sys, l++, _("[Playlist]"));
-    if (p_sys->b_color) color_set(C_DEFAULT, NULL);
-    MainBoxWrite(p_sys, l++, _("     r           Toggle Random playing"));
-    MainBoxWrite(p_sys, l++, _("     l           Toggle Loop Playlist"));
-    MainBoxWrite(p_sys, l++, _("     R           Toggle Repeat item"));
-    MainBoxWrite(p_sys, l++, _("     o           Order Playlist by title"));
-    MainBoxWrite(p_sys, l++, _("     O           Reverse order Playlist by title"));
-    MainBoxWrite(p_sys, l++, _("     g           Go to the current playing item"));
-    MainBoxWrite(p_sys, l++, _("     /           Look for an item"));
-    MainBoxWrite(p_sys, l++, _("     A           Add an entry"));
-    MainBoxWrite(p_sys, l++, _("     D, ⌫, <del> Delete an entry"));
-    MainBoxWrite(p_sys, l++, _("     e           Eject (if stopped)"));
-    MainBoxWrite(p_sys, l++, "");
-
-    if (p_sys->b_color) color_set(C_CATEGORY, NULL);
-    MainBoxWrite(p_sys, l++, _("[Filebrowser]"));
-    if (p_sys->b_color) color_set(C_DEFAULT, NULL);
-    MainBoxWrite(p_sys, l++, _("     <enter>     Add the selected file to the playlist"));
-    MainBoxWrite(p_sys, l++, _("     <space>     Add the selected directory to the playlist"));
-    MainBoxWrite(p_sys, l++, _("     .           Show/Hide hidden files"));
-    MainBoxWrite(p_sys, l++, "");
-
-    if (p_sys->b_color) color_set(C_CATEGORY, NULL);
-    MainBoxWrite(p_sys, l++, _("[Boxes]"));
-    if (p_sys->b_color) color_set(C_DEFAULT, NULL);
-    MainBoxWrite(p_sys, l++, _("     ↑,↓         Navigate through the box line by line"));
-    MainBoxWrite(p_sys, l++, _("     ⇞,⇟         Navigate through the box page by page"));
-    MainBoxWrite(p_sys, l++, _("     ↖,↘         Navigate to start/end of box"));
-    MainBoxWrite(p_sys, l++, "");
-
-    if (p_sys->b_color) color_set(C_CATEGORY, NULL);
-    MainBoxWrite(p_sys, l++, _("[Player]"));
-    if (p_sys->b_color) color_set(C_DEFAULT, NULL);
-    MainBoxWrite(p_sys, l++, _("     ↑,↓         Seek +/-5%%"));
-
+#define H(a) MainBoxWrite(sys, l++, a)
+
+    if (sys->color) color_set(C_CATEGORY, NULL);
+    H(_("[Display]"));
+    if (sys->color) color_set(C_DEFAULT, NULL);
+    H(_(" h,H                    Show/Hide help box"));
+    H(_(" i                      Show/Hide info box"));
+    H(_(" M                      Show/Hide metadata box"));
+    H(_(" L                      Show/Hide messages box"));
+    H(_(" P                      Show/Hide playlist box"));
+    H(_(" B                      Show/Hide filebrowser"));
+    H(_(" x                      Show/Hide objects box"));
+    H(_(" S                      Show/Hide statistics box"));
+    H(_(" Esc                    Close Add/Search entry"));
+    H(_(" Ctrl-l                 Refresh the screen"));
+    H("");
+
+    if (sys->color) color_set(C_CATEGORY, NULL);
+    H(_("[Global]"));
+    if (sys->color) color_set(C_DEFAULT, NULL);
+    H(_(" q, Q, Esc              Quit"));
+    H(_(" s                      Stop"));
+    H(_(" <space>                Pause/Play"));
+    H(_(" f                      Toggle Fullscreen"));
+    H(_(" c                      Cycle through audio tracks"));
+    H(_(" v                      Cycle through subtitles tracks"));
+    H(_(" b                      Cycle through video tracks"));
+    H(_(" n, p                   Next/Previous playlist item"));
+    H(_(" [, ]                   Next/Previous title"));
+    H(_(" <, >                   Next/Previous chapter"));
+    /* xgettext: You can use ← and → characters */
+    H(_(" <left>,<right>         Seek -/+ 1%%"));
+    H(_(" a, z                   Volume Up/Down"));
+    H(_(" m                      Mute"));
+    /* xgettext: You can use ↑ and ↓ characters */
+    H(_(" <up>,<down>            Navigate through the box line by line"));
+    /* xgettext: You can use ⇞ and ⇟ characters */
+    H(_(" <pageup>,<pagedown>    Navigate through the box page by page"));
+    /* xgettext: You can use ↖ and ↘ characters */
+    H(_(" <start>,<end>          Navigate to start/end of box"));
+    H("");
+
+    if (sys->color) color_set(C_CATEGORY, NULL);
+    H(_("[Playlist]"));
+    if (sys->color) color_set(C_DEFAULT, NULL);
+    H(_(" r                      Toggle Random playing"));
+    H(_(" l                      Toggle Loop Playlist"));
+    H(_(" R                      Toggle Repeat item"));
+    H(_(" o                      Order Playlist by title"));
+    H(_(" O                      Reverse order Playlist by title"));
+    H(_(" g                      Go to the current playing item"));
+    H(_(" /                      Look for an item"));
+    H(_(" ;                      Look for the next item"));
+    H(_(" A                      Add an entry"));
+    /* xgettext: You can use ⌫ character to translate <backspace> */
+    H(_(" D, <backspace>, <del>  Delete an entry"));
+    H(_(" e                      Eject (if stopped)"));
+    H("");
+
+    if (sys->color) color_set(C_CATEGORY, NULL);
+    H(_("[Filebrowser]"));
+    if (sys->color) color_set(C_DEFAULT, NULL);
+    H(_(" <enter>                Add the selected file to the playlist"));
+    H(_(" <space>                Add the selected directory to the playlist"));
+    H(_(" .                      Show/Hide hidden files"));
+    H("");
+
+    if (sys->color) color_set(C_CATEGORY, NULL);
+    H(_("[Player]"));
+    if (sys->color) color_set(C_DEFAULT, NULL);
+    /* xgettext: You can use ↑ and ↓ characters */
+    H(_(" <up>,<down>            Seek +/-5%%"));
+
+#undef H
     return l;
 }
 
-static int DrawBrowse(intf_thread_t *p_intf)
+static int DrawBrowse(intf_thread_t *intf)
 {
-    intf_sys_t *p_sys = p_intf->p_sys;
+    intf_sys_t *sys = intf->p_sys;
 
-    for(int i = 0; i < p_sys->i_dir_entries; i++)
-    {
-        struct dir_entry_t *p_dir_entry = p_sys->pp_dir_entries[i];
-        char type = p_dir_entry->b_file ? ' ' : '+';
+    for (int i = 0; i < sys->n_dir_entries; i++) {
+        struct dir_entry_t *dir_entry = sys->dir_entries[i];
+        char type = dir_entry->file ? ' ' : '+';
 
-        if (p_sys->b_color)
-            color_set(p_dir_entry->b_file ? C_DEFAULT : C_FOLDER, NULL);
-        MainBoxWrite(p_sys, i, " %c %s", type, p_dir_entry->psz_path);
+        if (sys->color)
+            color_set(dir_entry->file ? C_DEFAULT : C_FOLDER, NULL);
+        MainBoxWrite(sys, i, " %c %s", type, dir_entry->path);
     }
 
-    return p_sys->i_dir_entries;
+    return sys->n_dir_entries;
 }
 
-static int DrawPlaylist(intf_thread_t *p_intf)
+static int DrawPlaylist(intf_thread_t *intf)
 {
-    intf_sys_t *p_sys = p_intf->p_sys;
-    playlist_t *p_playlist = pl_Get(p_intf);
+    intf_sys_t *sys = intf->p_sys;
+    playlist_t *p_playlist = pl_Get(intf);
 
-    if (p_sys->b_need_update)
-    {
-        PlaylistRebuild(p_intf);
-        p_sys->b_need_update = false;
+    PL_LOCK;
+    vlc_mutex_lock(&sys->pl_lock);
+    if (sys->need_update) {
+        PlaylistRebuild(intf);
+        sys->need_update = false;
     }
+    vlc_mutex_unlock(&sys->pl_lock);
+    PL_UNLOCK;
 
-    if (p_sys->b_plidx_follow)
-        FindIndex(p_sys, p_playlist);
-
-    for(int i = 0; i < p_sys->i_plist_entries; i++)
-    {
-        char c = ' ';
-        playlist_item_t *p_current_item, *p_item, *p_node;
-        input_thread_t *p_input2;
+    if (sys->plidx_follow)
+        FindIndex(sys, p_playlist);
 
-        p_item = p_sys->pp_plist[i]->p_item;
-        p_node = p_sys->p_node;
-        p_input2 = playlist_CurrentInput(p_playlist);
+    for (int i = 0; i < sys->plist_entries; i++) {
+        char c;
+        playlist_item_t *current_item;
+        playlist_item_t *item = sys->plist[i]->item;
+        vlc_mutex_lock(&sys->pl_lock);
+        playlist_item_t *node = sys->node;
+        vlc_mutex_unlock(&sys->pl_lock);
 
         PL_LOCK;
-        assert(p_item);
-        p_current_item = playlist_CurrentPlayingItem(p_playlist);
-        if ((p_node && p_item->p_input == p_node->p_input) ||
-                    (!p_node && p_input2 && p_current_item &&
-                      p_item->p_input == p_current_item->p_input))
+        assert(item);
+        current_item = playlist_CurrentPlayingItem(p_playlist);
+        if ((node && item->p_input == node->p_input) ||
+           (!node && current_item && item->p_input == current_item->p_input))
             c = '*';
-        else if (p_item == p_node || (p_item != p_node &&
-                    PlaylistIsPlaying(p_playlist, p_item)))
+        else if (item == node || current_item == item)
             c = '>';
+        else
+            c = ' ';
         PL_UNLOCK;
 
-        if (p_input2)
-            vlc_object_release(p_input2);
-
-        if (p_sys->b_color) color_set(i%3 + C_PLAYLIST_1, NULL);
-        MainBoxWrite(p_sys, i, "%c%s", c, p_sys->pp_plist[i]->psz_display);
-        if (p_sys->b_color) color_set(C_DEFAULT, NULL);
+        if (sys->color) color_set(i%3 + C_PLAYLIST_1, NULL);
+        MainBoxWrite(sys, i, "%c%s", c, sys->plist[i]->display);
+        if (sys->color) color_set(C_DEFAULT, NULL);
     }
 
-    return p_sys->i_plist_entries;
+    return sys->plist_entries;
 }
 
-static int DrawMessages(intf_thread_t *p_intf)
+static int DrawMessages(intf_thread_t *intf)
 {
-    intf_sys_t *p_sys = p_intf->p_sys;
+    intf_sys_t *sys = intf->p_sys;
     int l = 0;
-    int i;
 
-    vlc_mutex_lock(&p_sys->msg_lock);
-    i = p_sys->i_msgs;
-    for(;;)
-    {
-        msg_item_t *msg = p_sys->msgs[i];
-        if (msg)
-        {
-            if (p_sys->b_color)
-                color_set(msg->i_type + C_INFO, NULL);
-            MainBoxWrite(p_sys, l++, "[%s] %s", msg->psz_module, msg->psz_msg);
+    vlc_mutex_lock(&sys->msg_lock);
+    int i = sys->i_msgs;
+    for(;;) {
+        vlc_log_t *msg = sys->msgs[i].item;
+        if (msg) {
+            if (sys->color)
+                color_set(sys->msgs[i].type + C_INFO, NULL);
+            MainBoxWrite(sys, l++, "[%s] %s", msg->psz_module, sys->msgs[i].msg);
         }
 
-        if (++i == sizeof p_sys->msgs / sizeof *p_sys->msgs)
+        if (++i == sizeof sys->msgs / sizeof *sys->msgs)
             i = 0;
 
-        if (i == p_sys->i_msgs) /* did we loop around the ring buffer ? */
+        if (i == sys->i_msgs) /* did we loop around the ring buffer ? */
             break;
     }
-    vlc_mutex_unlock(&p_sys->msg_lock);
+
+    vlc_mutex_unlock(&sys->msg_lock);
+    if (sys->color)
+        color_set(C_DEFAULT, NULL);
     return l;
 }
 
-static int DrawStatus(intf_thread_t *p_intf)
+static int DrawStatus(intf_thread_t *intf)
 {
-    intf_sys_t     *p_sys = p_intf->p_sys;
-    input_thread_t *p_input = p_sys->p_input;
-    playlist_t     *p_playlist = pl_Get(p_intf);
-    static const char name[] = "VLC media player "PACKAGE_VERSION;
-    const size_t name_len = sizeof name - 1; /* without \0 termination */
+    intf_sys_t     *sys = intf->p_sys;
+    input_thread_t *p_input = sys->p_input;
+    playlist_t     *p_playlist = pl_Get(intf);
+    char *name = _("VLC media player");
+    const size_t name_len = strlen(name) + sizeof(PACKAGE_VERSION);
     int y = 0;
     const char *repeat, *loop, *random;
 
@@ -1078,10 +1051,10 @@ static int DrawStatus(intf_thread_t *p_intf)
         padding = 0;
 
     attrset(A_REVERSE);
-    if (p_sys->b_color) color_set(C_TITLE, NULL);
+    if (sys->color) color_set(C_TITLE, NULL);
     DrawEmptyLine(y, 0, COLS);
-    mvnprintw(y++, padding / 2, COLS, "%s", name);
-    if (p_sys->b_color) color_set(C_STATUS, NULL);
+    mvnprintw(y++, padding / 2, COLS, "%s %s", name, PACKAGE_VERSION);
+    if (sys->color) color_set(C_STATUS, NULL);
     attroff(A_REVERSE);
 
     y++; /* leave a blank line */
@@ -1090,13 +1063,16 @@ static int DrawStatus(intf_thread_t *p_intf)
     random = var_GetBool(p_playlist, "random") ? _("[Random] ") : "";
     loop   = var_GetBool(p_playlist, "loop")   ? _("[Loop]")    : "";
 
-    if (p_input && !p_input->b_dead)
-    {
+    if (p_input && !p_input->b_dead) {
         vlc_value_t val;
+        char *path, *uri;
 
-        char *psz_uri = input_item_GetURI(input_GetItem(p_input));
-        mvnprintw(y++, 0, COLS, _(" Source   : %s"), psz_uri);
-        free(psz_uri);
+        uri = input_item_GetURI(input_GetItem(p_input));
+        path = make_path(uri);
+
+        mvnprintw(y++, 0, COLS, _(" Source   : %s"), path?path:uri);
+        free(uri);
+        free(path);
 
         var_Get(p_input, "state", &val);
         switch(val.i_int)
@@ -1108,7 +1084,7 @@ static int DrawStatus(intf_thread_t *p_intf)
             };
             char buf1[MSTRTIME_MAX_SIZE];
             char buf2[MSTRTIME_MAX_SIZE];
-            audio_volume_t i_volume;
+            float volume;
 
         case INIT_S:
         case END_S:
@@ -1129,37 +1105,35 @@ static int DrawStatus(intf_thread_t *p_intf)
 
             mvnprintw(y++, 0, COLS, _(" Position : %s/%s"), buf1, buf2);
 
-            aout_VolumeGet(p_playlist, &i_volume);
-            mvnprintw(y++, 0, COLS, _(" Volume   : %i%%"),
-                       i_volume*200/AOUT_VOLUME_MAX);
+            volume = playlist_VolumeGet(p_playlist);
+            int mute = playlist_MuteGet(p_playlist);
+            mvnprintw(y++, 0, COLS,
+                      mute ? _(" Volume   : Mute") :
+                      volume >= 0.f ? _(" Volume   : %3ld%%") : _(" Volume   : ----"),
+                      lroundf(volume * 100.f));
 
-            if (!var_Get(p_input, "title", &val))
-            {
+            if (!var_Get(p_input, "title", &val)) {
                 int i_title_count = var_CountChoices(p_input, "title");
                 if (i_title_count > 0)
                     mvnprintw(y++, 0, COLS, _(" Title    : %"PRId64"/%d"),
                                val.i_int, i_title_count);
             }
 
-            if (!var_Get(p_input, "chapter", &val))
-            {
+            if (!var_Get(p_input, "chapter", &val)) {
                 int i_chapter_count = var_CountChoices(p_input, "chapter");
-                if (i_chapter_count > 0)
-                    mvnprintw(y++, 0, COLS, _(" Chapter  : %"PRId64"/%d"),
+                if (i_chapter_count > 0) mvnprintw(y++, 0, COLS, _(" Chapter  : %"PRId64"/%d"),
                                val.i_int, i_chapter_count);
             }
         }
-    }
-    else
-    {
+    } else {
         mvnprintw(y++, 0, COLS, _(" Source: <no current item> "));
         mvnprintw(y++, 0, COLS, " %s%s%s", repeat, random, loop);
         mvnprintw(y++, 0, COLS, _(" [ h for help ]"));
         DrawEmptyLine(y++, 0, COLS);
     }
 
-    if (p_sys->b_color) color_set(C_DEFAULT, NULL);
-    DrawBox(y++, 1, p_sys->b_color, ""); /* position slider */
+    if (sys->color) color_set(C_DEFAULT, NULL);
+    DrawBox(y++, 1, sys->color, ""); /* position slider */
     DrawEmptyLine(y, 1, COLS-2);
     if (p_input)
         DrawLine(y, 1, (int)((COLS-2) * var_GetFloat(p_input, "position")));
@@ -1169,21 +1143,20 @@ static int DrawStatus(intf_thread_t *p_intf)
     return y;
 }
 
-static void FillTextBox(intf_sys_t *p_sys)
+static void FillTextBox(intf_sys_t *sys)
 {
     int width = COLS - 2;
-    const char *title = p_sys->i_box_type == BOX_OPEN ? "Open: %s" : "Find: %s";
-    char *chain = p_sys->i_box_type == BOX_OPEN ? p_sys->psz_open_chain :
-                    p_sys->psz_old_search ?  p_sys->psz_old_search :
-                     p_sys->psz_search_chain;
 
     DrawEmptyLine(7, 1, width);
-    mvnprintw(7, 1, width, _(title), chain);
+    if (sys->box_type == BOX_OPEN)
+        mvnprintw(7, 1, width, _("Open: %s"), sys->open_chain);
+    else
+        mvnprintw(7, 1, width, _("Find: %s"), sys->search_chain);
 }
 
-static void FillBox(intf_thread_t *p_intf)
+static void FillBox(intf_thread_t *intf)
 {
-    intf_sys_t *p_sys = p_intf->p_sys;
+    intf_sys_t *sys = intf->p_sys;
     static int (* const draw[]) (intf_thread_t *) = {
         [BOX_HELP]      = DrawHelp,
         [BOX_INFO]      = DrawInfo,
@@ -1197,44 +1170,44 @@ static void FillBox(intf_thread_t *p_intf)
         [BOX_LOG]       = DrawMessages,
     };
 
-    p_sys->i_box_lines_total = draw[p_sys->i_box_type](p_intf);
+    sys->box_lines_total = draw[sys->box_type](intf);
 
-    if (p_sys->i_box_type == BOX_SEARCH || p_sys->i_box_type == BOX_OPEN)
-        FillTextBox(p_sys);
+    if (sys->box_type == BOX_SEARCH || sys->box_type == BOX_OPEN)
+        FillTextBox(sys);
 }
 
-static void Redraw(intf_thread_t *p_intf, time_t *t_last_refresh)
+static void Redraw(intf_thread_t *intf)
 {
-    intf_sys_t *p_sys   = p_intf->p_sys;
-    int         box     = p_sys->i_box_type;
-    int         y       = DrawStatus(p_intf);
+    intf_sys_t *sys   = intf->p_sys;
+    int         box     = sys->box_type;
+    int         y       = DrawStatus(intf);
 
-    p_sys->i_box_height = LINES - y - 2;
-    DrawBox(y++, p_sys->i_box_height, p_sys->b_color, _(box_title[box]));
+    sys->box_height = LINES - y - 2;
+    DrawBox(y++, sys->box_height, sys->color, _(box_title[box]));
 
-    p_sys->i_box_y = y;
+    sys->box_y = y;
 
-    if (box != BOX_NONE)
-    {
-        FillBox(p_intf);
+    if (box != BOX_NONE) {
+        FillBox(intf);
 
-        if (p_sys->i_box_start > p_sys->i_box_lines_total - 1)
-            p_sys->i_box_start = p_sys->i_box_lines_total - 1;
-        y += __MIN(p_sys->i_box_lines_total - p_sys->i_box_start,
-                   p_sys->i_box_height);
+        if (sys->box_lines_total == 0)
+            sys->box_start = 0;
+        else if (sys->box_start > sys->box_lines_total - 1)
+            sys->box_start = sys->box_lines_total - 1;
+        y += __MIN(sys->box_lines_total - sys->box_start,
+                   sys->box_height);
     }
 
     while (y < LINES - 1)
         DrawEmptyLine(y++, 1, COLS - 2);
 
     refresh();
-    *t_last_refresh = time(0);
 }
 
-static void ChangePosition(intf_thread_t *p_intf, float increment)
+static void ChangePosition(intf_thread_t *intf, float increment)
 {
-    intf_sys_t     *p_sys = p_intf->p_sys;
-    input_thread_t *p_input = p_sys->p_input;
+    intf_sys_t     *sys = intf->p_sys;
+    input_thread_t *p_input = sys->p_input;
     float pos;
 
     if (!p_input || var_GetInteger(p_input, "state") != PLAYING_S)
@@ -1255,7 +1228,7 @@ static inline void RemoveLastUTF8Entity(char *psz, int len)
     psz[len] = '\0';
 }
 
-static char *GetDiscDevice(intf_thread_t *p_intf, const char *name)
+static char *GetDiscDevice(intf_thread_t *intf, const char *name)
 {
     static const struct { const char *s; size_t n; const char *v; } devs[] =
     {
@@ -1265,19 +1238,14 @@ static char *GetDiscDevice(intf_thread_t *p_intf, const char *name)
     };
     char *device;
 
-    for (unsigned i = 0; i < sizeof devs / sizeof *devs; i++)
-    {
+    for (unsigned i = 0; i < sizeof devs / sizeof *devs; i++) {
         size_t n = devs[i].n;
-        if (!strncmp(name, devs[i].s, n))
-            switch(name[n])
-            {
-            case '\0':
-            case '@':
-                return config_GetPsz(p_intf, devs[i].v);
-            default:
-                /* Omit the beginning MRL-selector characters */
-                return strdup(name + n);
-            }
+        if (!strncmp(name, devs[i].s, n)) {
+            if (name[n] == '@' || name[n] == '\0')
+                return config_GetPsz(intf, devs[i].v);
+            /* Omit the beginning MRL-selector characters */
+            return strdup(name + n);
+        }
     }
 
     device = strdup(name);
@@ -1288,60 +1256,56 @@ static char *GetDiscDevice(intf_thread_t *p_intf, const char *name)
     return device;
 }
 
-static void Eject(intf_thread_t *p_intf)
+static void Eject(intf_thread_t *intf)
 {
-    char *psz_device, *psz_name;
-    playlist_t * p_playlist = pl_Get(p_intf);
+    char *device, *name;
+    playlist_t * p_playlist = pl_Get(intf);
 
     /* If there's a stream playing, we aren't allowed to eject ! */
-    if (p_intf->p_sys->p_input)
+    if (intf->p_sys->p_input)
         return;
 
     PL_LOCK;
 
-    if (!playlist_CurrentPlayingItem(p_playlist))
-    {
+    if (!playlist_CurrentPlayingItem(p_playlist)) {
         PL_UNLOCK;
         return;
     }
 
-    psz_name = playlist_CurrentPlayingItem(p_playlist)->p_input->psz_name;
-    psz_device = psz_name ? GetDiscDevice(p_intf, psz_name) : NULL;
+    name = playlist_CurrentPlayingItem(p_playlist)->p_input->psz_name;
+    device = name ? GetDiscDevice(intf, name) : NULL;
 
     PL_UNLOCK;
 
-    if (psz_device)
-    {
-        intf_Eject(p_intf, psz_device);
-        free(psz_device);
+    if (device) {
+        intf_Eject(intf, device);
+        free(device);
     }
 }
 
-static void PlayPause(intf_thread_t *p_intf)
+static void PlayPause(intf_thread_t *intf)
 {
-    input_thread_t *p_input = p_intf->p_sys->p_input;
+    input_thread_t *p_input = intf->p_sys->p_input;
 
-    if (p_input)
-    {
+    if (p_input) {
         int64_t state = var_GetInteger( p_input, "state" );
         state = (state != PLAYING_S) ? PLAYING_S : PAUSE_S;
         var_SetInteger( p_input, "state", state );
-    }
-    else
-        playlist_Play(pl_Get(p_intf));
+    } else
+        playlist_Play(pl_Get(intf));
 }
 
-static inline void BoxSwitch(intf_sys_t *p_sys, int box)
+static inline void BoxSwitch(intf_sys_t *sys, int box)
 {
-    p_sys->i_box_type = (p_sys->i_box_type == box) ? BOX_NONE : box;
-    p_sys->i_box_start = 0;
-    p_sys->i_box_idx = 0;
+    sys->box_type = (sys->box_type == box) ? BOX_NONE : box;
+    sys->box_start = 0;
+    sys->box_idx = 0;
 }
 
-static bool HandlePlaylistKey(intf_thread_t *p_intf, int key)
+static bool HandlePlaylistKey(intf_thread_t *intf, int key)
 {
-    intf_sys_t *p_sys = p_intf->p_sys;
-    playlist_t *p_playlist = pl_Get(p_intf);
+    intf_sys_t *sys = intf->p_sys;
+    playlist_t *p_playlist = pl_Get(intf);
     struct pl_item_t *p_pl_item;
 
     switch(key)
@@ -1357,11 +1321,17 @@ static bool HandlePlaylistKey(intf_thread_t *p_intf, int key)
         playlist_RecursiveNodeSort(p_playlist, p_playlist->p_root_onelevel,
                                     SORT_TITLE_NODES_FIRST,
                                     (key == 'o')? ORDER_NORMAL : ORDER_REVERSE);
-        p_sys->b_need_update = true;
+        vlc_mutex_lock(&sys->pl_lock);
+        sys->need_update = true;
+        vlc_mutex_unlock(&sys->pl_lock);
+        return true;
+
+    case ';':
+        SearchPlaylist(sys);
         return true;
 
     case 'g':
-        FindIndex(p_sys, p_playlist);
+        FindIndex(sys, p_playlist);
         return true;
 
     /* Deletion */
@@ -1370,108 +1340,105 @@ static bool HandlePlaylistKey(intf_thread_t *p_intf, int key)
     case 0x7f:
     case KEY_DC:
     {
-        playlist_item_t *p_item;
+        playlist_item_t *item;
 
         PL_LOCK;
-        p_item = p_sys->pp_plist[p_sys->i_box_idx]->p_item;
-        if (p_item->i_children == -1)
-            playlist_DeleteFromInput(p_playlist, p_item->p_input, pl_Locked);
+        item = sys->plist[sys->box_idx]->item;
+        if (item->i_children == -1)
+            playlist_DeleteFromInput(p_playlist, item->p_input, pl_Locked);
         else
-            playlist_NodeDelete(p_playlist, p_item, true , false);
+            playlist_NodeDelete(p_playlist, item, true , false);
         PL_UNLOCK;
-        p_sys->b_need_update = true;
+        vlc_mutex_lock(&sys->pl_lock);
+        if (sys->box_idx >= sys->box_lines_total - 1)
+            sys->box_idx = sys->box_lines_total - 2;
+        sys->need_update = true;
+        vlc_mutex_unlock(&sys->pl_lock);
         return true;
     }
 
     case KEY_ENTER:
     case '\r':
     case '\n':
-        if (!(p_pl_item = p_sys->pp_plist[p_sys->i_box_idx]))
+        if (!(p_pl_item = sys->plist[sys->box_idx]))
             return false;
 
-        if (p_pl_item->p_item->i_children)
-        {
-            playlist_item_t *p_item, *p_parent = p_pl_item->p_item;
-            if (p_parent->i_children == -1)
-            {
-                p_item = p_parent;
+        if (p_pl_item->item->i_children) {
+            playlist_item_t *item, *p_parent = p_pl_item->item;
+            if (p_parent->i_children == -1) {
+                item = p_parent;
 
                 while (p_parent->p_parent)
                     p_parent = p_parent->p_parent;
-            }
-            else
-            {
-                p_sys->p_node = p_parent;
-                p_item = NULL;
+            } else {
+                vlc_mutex_lock(&sys->pl_lock);
+                sys->node = p_parent;
+                vlc_mutex_unlock(&sys->pl_lock);
+                item = NULL;
             }
 
             playlist_Control(p_playlist, PLAYLIST_VIEWPLAY, pl_Unlocked,
-                              p_parent, p_item);
-        }
-        else
-        {   /* We only want to set the current node */
+                              p_parent, item);
+        } else {   /* We only want to set the current node */
             playlist_Stop(p_playlist);
-            p_sys->p_node = p_pl_item->p_item;
+            vlc_mutex_lock(&sys->pl_lock);
+            sys->node = p_pl_item->item;
+            vlc_mutex_unlock(&sys->pl_lock);
         }
 
-        p_sys->b_plidx_follow = true;
+        sys->plidx_follow = true;
         return true;
     }
 
     return false;
 }
 
-static bool HandleBrowseKey(intf_thread_t *p_intf, int key)
+static bool HandleBrowseKey(intf_thread_t *intf, int key)
 {
-    intf_sys_t *p_sys = p_intf->p_sys;
+    intf_sys_t *sys = intf->p_sys;
     struct dir_entry_t *dir_entry;
 
     switch(key)
     {
     case '.':
-        p_sys->b_show_hidden_files = !p_sys->b_show_hidden_files;
-        ReadDir(p_intf);
+        sys->show_hidden_files = !sys->show_hidden_files;
+        ReadDir(intf);
         return true;
 
     case KEY_ENTER:
     case '\r':
     case '\n':
     case ' ':
-        dir_entry = p_sys->pp_dir_entries[p_sys->i_box_idx];
+        dir_entry = sys->dir_entries[sys->box_idx];
+        char *path;
+        if (asprintf(&path, "%s" DIR_SEP "%s", sys->current_dir,
+                     dir_entry->path) == -1)
+            return true;
 
-        if (!dir_entry->b_file && key != ' ')
-        {
-            char *current_dir = p_sys->psz_current_dir;
-            if (asprintf(&p_sys->psz_current_dir, "%s/%s",
-                          p_sys->psz_current_dir, dir_entry->psz_path) != -1)
-            {
-                ReadDir(p_intf);
-                free(current_dir);
-            }
-            else
-                p_sys->psz_current_dir = current_dir;
+        if (!dir_entry->file && key != ' ') {
+            free(sys->current_dir);
+            sys->current_dir = path;
+            ReadDir(intf);
 
-            p_sys->i_box_start = 0;
-            p_sys->i_box_idx = 0;
+            sys->box_start = 0;
+            sys->box_idx = 0;
             return true;
         }
 
-        char* psz_uri;
-        if (asprintf(&psz_uri, "%s://%s/%s",
-                    dir_entry->b_file ? "file" : "directory",
-                    p_sys->psz_current_dir, dir_entry->psz_path) == -1)
-        {
-            return false;
-        }
+        char *uri = vlc_path2uri(path, "file");
+        free(path);
+        if (uri == NULL)
+            return true;
 
-        playlist_t *p_playlist = pl_Get(p_intf);
-        playlist_item_t *p_parent = p_sys->p_node;
-        if (!p_parent)
-        {
-            playlist_item_t *p_item;
+        playlist_t *p_playlist = pl_Get(intf);
+        vlc_mutex_lock(&sys->pl_lock);
+        playlist_item_t *p_parent = sys->node;
+        vlc_mutex_unlock(&sys->pl_lock);
+        if (!p_parent) {
+            playlist_item_t *item;
             PL_LOCK;
-            p_item = playlist_CurrentPlayingItem(p_playlist);
-            p_parent = p_item ? p_item->p_parent : NULL;
+            item = playlist_CurrentPlayingItem(p_playlist);
+            p_parent = item ? item->p_parent : NULL;
             PL_UNLOCK;
             if (!p_parent)
                 p_parent = p_playlist->p_local_onelevel;
@@ -1481,22 +1448,56 @@ static bool HandleBrowseKey(intf_thread_t *p_intf, int key)
             p_parent = p_parent->p_parent;
 
         input_item_t *p_input = p_playlist->p_local_onelevel->p_input;
-        playlist_Add(p_playlist, psz_uri, NULL, PLAYLIST_APPEND,
+        playlist_Add(p_playlist, uri, NULL, PLAYLIST_APPEND,
                       PLAYLIST_END, p_parent->p_input == p_input, false);
 
-        p_sys->i_box_type = BOX_PLAYLIST;
-        free(psz_uri);
+        BoxSwitch(sys, BOX_PLAYLIST);
+        free(uri);
         return true;
     }
 
     return false;
 }
 
-static bool HandleEditBoxKey(intf_thread_t *p_intf, int key, int box)
+static void OpenSelection(intf_thread_t *intf)
 {
-    intf_sys_t *p_sys = p_intf->p_sys;
+    intf_sys_t *sys = intf->p_sys;
+    char *uri = vlc_path2uri(sys->open_chain, NULL);
+    if (uri == NULL)
+        return;
+
+    playlist_t *p_playlist = pl_Get(intf);
+    vlc_mutex_lock(&sys->pl_lock);
+    playlist_item_t *p_parent = sys->node;
+    vlc_mutex_unlock(&sys->pl_lock);
+
+    PL_LOCK;
+    if (!p_parent) {
+        playlist_item_t *current;
+        current= playlist_CurrentPlayingItem(p_playlist);
+        p_parent = current ? current->p_parent : NULL;
+        if (!p_parent)
+            p_parent = p_playlist->p_local_onelevel;
+    }
+
+    while (p_parent->p_parent && p_parent->p_parent->p_parent)
+        p_parent = p_parent->p_parent;
+    PL_UNLOCK;
+
+    playlist_Add(p_playlist, uri, NULL,
+            PLAYLIST_APPEND|PLAYLIST_GO, PLAYLIST_END,
+            p_parent->p_input == p_playlist->p_local_onelevel->p_input,
+            false);
+
+    sys->plidx_follow = true;
+    free(uri);
+}
+
+static void HandleEditBoxKey(intf_thread_t *intf, int key, int box)
+{
+    intf_sys_t *sys = intf->p_sys;
     bool search = box == BOX_SEARCH;
-    char *str = search ? p_sys->psz_search_chain: p_sys->psz_open_chain;
+    char *str = search ? sys->search_chain: sys->open_chain;
     size_t len = strlen(str);
 
     assert(box == BOX_SEARCH || box == BOX_OPEN);
@@ -1504,51 +1505,24 @@ static bool HandleEditBoxKey(intf_thread_t *p_intf, int key, int box)
     switch(key)
     {
     case 0x0c:  /* ^l */
-    case KEY_CLEAR:     clear(); return 1;
+    case KEY_CLEAR:     clear(); return;
 
     case KEY_ENTER:
     case '\r':
     case '\n':
         if (search)
-        {
-            if (len)
-                p_sys->psz_old_search = strdup(p_sys->psz_search_chain);
-            else if (p_sys->psz_old_search)
-                SearchPlaylist(p_sys, p_sys->psz_old_search);
-        }
-        else if (len)
-        {
-            playlist_t *p_playlist = pl_Get(p_intf);
-            playlist_item_t *p_parent = p_sys->p_node, *p_current;
-
-            PL_LOCK;
-            if (!p_parent)
-            {
-                p_current = playlist_CurrentPlayingItem(p_playlist);
-                p_parent = p_current ? p_current->p_parent : NULL;
-                if (!p_parent)
-                    p_parent = p_playlist->p_local_onelevel;
-            }
-
-            while (p_parent->p_parent && p_parent->p_parent->p_parent)
-                p_parent = p_parent->p_parent;
-            PL_UNLOCK;
-
-            playlist_Add(p_playlist, p_sys->psz_open_chain, NULL,
-                  PLAYLIST_APPEND|PLAYLIST_GO, PLAYLIST_END,
-                  p_parent->p_input == p_playlist->p_local_onelevel->p_input,
-                  false);
+            SearchPlaylist(sys);
+        else
+            OpenSelection(intf);
 
-            p_sys->b_plidx_follow = true;
-        }
-        p_sys->i_box_type = BOX_PLAYLIST;
-        return 1;
+        sys->box_type = BOX_PLAYLIST;
+        return;
 
     case 0x1b: /* ESC */
         /* Alt+key combinations return 2 keys in the terminal keyboard:
          * ESC, and the 2nd key.
          * If some other key is available immediately (where immediately
-         * means after wgetch() 1 second delay), that means that the
+         * means after getch() 1 second delay), that means that the
          * ESC key was not pressed.
          *
          * man 3X curs_getch says:
@@ -1559,12 +1533,9 @@ static bool HandleEditBoxKey(intf_thread_t *p_intf, int key, int box)
          * following function-key sequence.
          *
          */
-        if (getch() != ERR)
-            return 0;
-
-        if (search) p_sys->i_box_idx = p_sys->i_before_search;
-        p_sys->i_box_type = BOX_PLAYLIST;
-        return 1;
+        if (getch() == ERR)
+            sys->box_type = BOX_PLAYLIST;
+        return;
 
     case KEY_BACKSPACE:
     case 0x7f:
@@ -1572,21 +1543,15 @@ static bool HandleEditBoxKey(intf_thread_t *p_intf, int key, int box)
         break;
 
     default:
-        if (len + 1 < search ? sizeof p_sys->psz_search_chain
-                             : sizeof p_sys->psz_open_chain)
-        {
-            str[len + 0] = (char) key;
+        if (len + 1 < (search ? sizeof sys->search_chain
+                              : sizeof sys->open_chain)) {
+            str[len + 0] = key;
             str[len + 1] = '\0';
         }
     }
 
     if (search)
-    {
-        free(p_sys->psz_old_search);
-        p_sys->psz_old_search = NULL;
-        SearchPlaylist(p_sys, str);
-    }
-    return 1;
+        SearchPlaylist(sys);
 }
 
 static void InputNavigate(input_thread_t* p_input, const char *var)
@@ -1595,131 +1560,160 @@ static void InputNavigate(input_thread_t* p_input, const char *var)
         var_TriggerCallback(p_input, var);
 }
 
-static bool HandleCommonKey(intf_thread_t *p_intf, int key)
+static void CycleESTrack(intf_sys_t *sys, const char *var)
+{
+    input_thread_t *input = sys->p_input;
+
+    if (!input)
+        return;
+
+    vlc_value_t val;
+    if (var_Change(input, var, VLC_VAR_GETCHOICES, &val, NULL) < 0)
+        return;
+
+    vlc_list_t *list = val.p_list;
+    int64_t current = var_GetInteger(input, var);
+
+    int i;
+    for (i = 0; i < list->i_count; i++)
+        if (list->p_values[i].i_int == current)
+            break;
+
+    if (++i >= list->i_count)
+        i = 0;
+    var_SetInteger(input, var, list->p_values[i].i_int);
+}
+
+static void HandleCommonKey(intf_thread_t *intf, int key)
 {
-    intf_sys_t *p_sys = p_intf->p_sys;
-    playlist_t *p_playlist = pl_Get(p_intf);
+    intf_sys_t *sys = intf->p_sys;
+    playlist_t *p_playlist = pl_Get(intf);
     switch(key)
     {
     case 0x1b:  /* ESC */
         if (getch() != ERR)
-            return false;
+            return;
 
     case 'q':
     case 'Q':
     case KEY_EXIT:
-        libvlc_Quit(p_intf->p_libvlc);
-        return false;
+        libvlc_Quit(intf->p_libvlc);
+        sys->exit = true;           // terminate the main loop
+        return;
 
     case 'h':
-    case 'H': BoxSwitch(p_sys, BOX_HELP);       return true;
-    case 'i': BoxSwitch(p_sys, BOX_INFO);       return true;
-    case 'm': BoxSwitch(p_sys, BOX_META);       return true;
-    case 'L': BoxSwitch(p_sys, BOX_LOG);        return true;
-    case 'P': BoxSwitch(p_sys, BOX_PLAYLIST);   return true;
-    case 'B': BoxSwitch(p_sys, BOX_BROWSE);     return true;
-    case 'x': BoxSwitch(p_sys, BOX_OBJECTS);    return true;
-    case 'S': BoxSwitch(p_sys, BOX_STATS);      return true;
+    case 'H': BoxSwitch(sys, BOX_HELP);       return;
+    case 'i': BoxSwitch(sys, BOX_INFO);       return;
+    case 'M': BoxSwitch(sys, BOX_META);       return;
+    case 'L': BoxSwitch(sys, BOX_LOG);        return;
+    case 'P': BoxSwitch(sys, BOX_PLAYLIST);   return;
+    case 'B': BoxSwitch(sys, BOX_BROWSE);     return;
+    case 'x': BoxSwitch(sys, BOX_OBJECTS);    return;
+    case 'S': BoxSwitch(sys, BOX_STATS);      return;
 
     case '/': /* Search */
-        p_sys->psz_search_chain[0] = '\0';
-        p_sys->b_plidx_follow = false;
-        p_sys->i_before_search = p_sys->i_box_idx;
-        p_sys->i_box_type = BOX_SEARCH;
-        return true;
+        sys->plidx_follow = false;
+        BoxSwitch(sys, BOX_SEARCH);
+        return;
 
     case 'A': /* Open */
-        p_sys->psz_open_chain[0] = '\0';
-        p_sys->i_box_type = BOX_OPEN;
-        return true;
+        sys->open_chain[0] = '\0';
+        BoxSwitch(sys, BOX_OPEN);
+        return;
 
     /* Navigation */
-    case KEY_RIGHT: ChangePosition(p_intf, +0.01); return true;
-    case KEY_LEFT:  ChangePosition(p_intf, -0.01); return true;
+    case KEY_RIGHT: ChangePosition(intf, +0.01); return;
+    case KEY_LEFT:  ChangePosition(intf, -0.01); return;
 
     /* Common control */
     case 'f':
-        if (p_sys->p_input)
-        {
-            vout_thread_t *p_vout = input_GetVout(p_sys->p_input);
-            if (p_vout)
-            {
+        if (sys->p_input) {
+            vout_thread_t *p_vout = input_GetVout(sys->p_input);
+            if (p_vout) {
                 bool fs = var_ToggleBool(p_playlist, "fullscreen");
                 var_SetBool(p_vout, "fullscreen", fs);
                 vlc_object_release(p_vout);
             }
         }
-        return false;
+        return;
 
-    case ' ': PlayPause(p_intf);            return true;
-    case 's': playlist_Stop(p_playlist);    return true;
-    case 'e': Eject(p_intf);                return true;
+    case ' ': PlayPause(intf);            return;
+    case 's': playlist_Stop(p_playlist);    return;
+    case 'e': Eject(intf);                return;
 
-    case '[': InputNavigate(p_sys->p_input, "prev-title");      return true;
-    case ']': InputNavigate(p_sys->p_input, "next-title");      return true;
-    case '<': InputNavigate(p_sys->p_input, "prev-chapter");    return true;
-    case '>': InputNavigate(p_sys->p_input, "next-chapter");    return true;
+    case '[': InputNavigate(sys->p_input, "prev-title");      return;
+    case ']': InputNavigate(sys->p_input, "next-title");      return;
+    case '<': InputNavigate(sys->p_input, "prev-chapter");    return;
+    case '>': InputNavigate(sys->p_input, "next-chapter");    return;
 
     case 'p': playlist_Prev(p_playlist);            break;
     case 'n': playlist_Next(p_playlist);            break;
-    case 'a': aout_VolumeUp(p_playlist, 1, NULL);   break;
-    case 'z': aout_VolumeDown(p_playlist, 1, NULL); break;
+    case 'a': playlist_VolumeUp(p_playlist, 1, NULL);   break;
+    case 'z': playlist_VolumeDown(p_playlist, 1, NULL); break;
+    case 'm': playlist_MuteToggle(p_playlist); break;
+
+    case 'c': CycleESTrack(sys, "audio-es"); break;
+    case 'v': CycleESTrack(sys, "spu-es");   break;
+    case 'b': CycleESTrack(sys, "video-es"); break;
 
     case 0x0c:  /* ^l */
     case KEY_CLEAR:
         break;
 
     default:
-        return false;
+        return;
     }
 
     clear();
-    return true;
+    return;
 }
 
-static bool HandleListKey(intf_thread_t *p_intf, int key)
+static bool HandleListKey(intf_thread_t *intf, int key)
 {
-    intf_sys_t *p_sys = p_intf->p_sys;
-    playlist_t *p_playlist = pl_Get(p_intf);
+    intf_sys_t *sys = intf->p_sys;
+    playlist_t *p_playlist = pl_Get(intf);
 
     switch(key)
     {
 #ifdef __FreeBSD__
+/* workaround for FreeBSD + xterm:
+ * see http://www.nabble.com/curses-vs.-xterm-key-mismatch-t3574377.html */
     case KEY_SELECT:
 #endif
-    case KEY_END:  p_sys->i_box_idx = p_sys->i_box_lines_total - 1; break;
-    case KEY_HOME: p_sys->i_box_idx = 0;                            break;
-    case KEY_UP:   p_sys->i_box_idx--;                              break;
-    case KEY_DOWN: p_sys->i_box_idx++;                              break;
-    case KEY_PPAGE:p_sys->i_box_idx -= p_sys->i_box_height;         break;
-    case KEY_NPAGE:p_sys->i_box_idx += p_sys->i_box_height;         break;
+    case KEY_END:  sys->box_idx = sys->box_lines_total - 1; break;
+    case KEY_HOME: sys->box_idx = 0;                            break;
+    case KEY_UP:   sys->box_idx--;                              break;
+    case KEY_DOWN: sys->box_idx++;                              break;
+    case KEY_PPAGE:sys->box_idx -= sys->box_height;         break;
+    case KEY_NPAGE:sys->box_idx += sys->box_height;         break;
     default:
         return false;
     }
 
-    CheckIdx(p_sys);
+    CheckIdx(sys);
 
-    if (p_sys->i_box_type == BOX_PLAYLIST)
-    {
+    if (sys->box_type == BOX_PLAYLIST) {
         PL_LOCK;
-        p_sys->b_plidx_follow = IsIndex(p_sys, p_playlist, p_sys->i_box_idx);
+        sys->plidx_follow = IsIndex(sys, p_playlist, sys->box_idx);
         PL_UNLOCK;
     }
 
     return true;
 }
 
-static bool HandleKey(intf_thread_t *p_intf)
+static void HandleKey(intf_thread_t *intf)
 {
-    intf_sys_t *p_sys = p_intf->p_sys;
+    intf_sys_t *sys = intf->p_sys;
     int key = getch();
-    int box = p_sys->i_box_type;
+    int box = sys->box_type;
 
     if (key == -1)
-        return false;
+        return;
 
-    if (box == BOX_SEARCH || box == BOX_OPEN)
-        return HandleEditBoxKey(p_intf, key, p_sys->i_box_type);
+    if (box == BOX_SEARCH || box == BOX_OPEN) {
+        HandleEditBoxKey(intf, key, sys->box_type);
+        return;
+    }
 
     if (box == BOX_NONE)
         switch(key)
@@ -1727,87 +1721,112 @@ static bool HandleKey(intf_thread_t *p_intf)
 #ifdef __FreeBSD__
         case KEY_SELECT:
 #endif
-        case KEY_END:   ChangePosition(p_intf, +.99);   return true;
-        case KEY_HOME:  ChangePosition(p_intf, -1.0);   return true;
-        case KEY_UP:    ChangePosition(p_intf, +0.05);  return true;
-        case KEY_DOWN:  ChangePosition(p_intf, -0.05);  return true;
-        default:        return HandleCommonKey(p_intf, key);
+        case KEY_END:   ChangePosition(intf, +.99);   return;
+        case KEY_HOME:  ChangePosition(intf, -1.0);   return;
+        case KEY_UP:    ChangePosition(intf, +0.05);  return;
+        case KEY_DOWN:  ChangePosition(intf, -0.05);  return;
+        default:        HandleCommonKey(intf, key);   return;
         }
 
-    if (box == BOX_BROWSE   && HandleBrowseKey(p_intf, key))
-        return true;
+    if (box == BOX_BROWSE   && HandleBrowseKey(intf, key))
+        return;
 
-    if (box == BOX_PLAYLIST && HandlePlaylistKey(p_intf, key))
-        return true;
+    if (box == BOX_PLAYLIST && HandlePlaylistKey(intf, key))
+        return;
 
-    return HandleListKey(p_intf, key) || HandleCommonKey(p_intf, key);
+    if (HandleListKey(intf, key))
+        return;
+
+    HandleCommonKey(intf, key);
 }
 
 /*
  *
  */
+static vlc_log_t *msg_Copy (const vlc_log_t *msg)
+{
+    vlc_log_t *copy = (vlc_log_t *)xmalloc (sizeof (*copy));
+    copy->i_object_id = msg->i_object_id;
+    copy->psz_object_type = msg->psz_object_type;
+    copy->psz_module = strdup (msg->psz_module);
+    copy->psz_header = msg->psz_header ? strdup (msg->psz_header) : NULL;
+    return copy;
+}
 
-static void MsgCallback(msg_cb_data_t *data, msg_item_t *msg, unsigned i)
+static void msg_Free (vlc_log_t *msg)
 {
-    intf_sys_t *p_sys = data->p_sys;
-    (void)i; // what is this?
-    int canc = vlc_savecancel();
+    free ((char *)msg->psz_module);
+    free ((char *)msg->psz_header);
+    free (msg);
+}
+
+static void MsgCallback(void *data, int type, const vlc_log_t *msg,
+                        const char *format, va_list ap)
+{
+    intf_sys_t *sys = data;
+    char *text;
 
-    vlc_mutex_lock(&p_sys->msg_lock);
+    if (sys->verbosity < 0
+     || sys->verbosity < (type - VLC_MSG_ERR)
+     || vasprintf(&text, format, ap) == -1)
+        return;
 
-    if (p_sys->msgs[p_sys->i_msgs])
-        msg_Release(p_sys->msgs[p_sys->i_msgs]);
-    p_sys->msgs[p_sys->i_msgs++] = msg_Hold(msg);
+    vlc_mutex_lock(&sys->msg_lock);
 
-    if (p_sys->i_msgs == (sizeof p_sys->msgs / sizeof *p_sys->msgs))
-        p_sys->i_msgs = 0;
+    sys->msgs[sys->i_msgs].type = type;
+    if (sys->msgs[sys->i_msgs].item != NULL)
+        msg_Free(sys->msgs[sys->i_msgs].item);
+    sys->msgs[sys->i_msgs].item = msg_Copy(msg);
+    free(sys->msgs[sys->i_msgs].msg);
+    sys->msgs[sys->i_msgs].msg = text;
+
+    if (++sys->i_msgs == (sizeof sys->msgs / sizeof *sys->msgs))
+        sys->i_msgs = 0;
+
+    vlc_mutex_unlock(&sys->msg_lock);
+}
 
-    vlc_mutex_unlock(&p_sys->msg_lock);
+static inline void UpdateInput(intf_sys_t *sys, playlist_t *p_playlist)
+{
+    if (!sys->p_input) {
+        sys->p_input = playlist_CurrentInput(p_playlist);
+    } else if (sys->p_input->b_dead) {
+        vlc_object_release(sys->p_input);
+        sys->p_input = NULL;
+    }
+}
 
-    vlc_restorecancel(canc);
+static void cleanup_run(void *data)
+{
+    intf_thread_t *intf = data;
+    playlist_t *p_playlist = pl_Get(intf);
+    var_DelCallback(p_playlist, "intf-change", PlaylistChanged, intf);
+    var_DelCallback(p_playlist, "item-change", ItemChanged, intf);
+    var_DelCallback(p_playlist, "playlist-item-append", PlaylistChanged, intf);
 }
 
 /*****************************************************************************
  * Run: ncurses thread
  *****************************************************************************/
-static void Run(intf_thread_t *p_intf)
+static void *Run(void *data)
 {
-    intf_sys_t    *p_sys = p_intf->p_sys;
-    playlist_t    *p_playlist = pl_Get(p_intf);
-
-    time_t t_last_refresh;
-    int canc = vlc_savecancel();
-
-    Redraw(p_intf, &t_last_refresh);
-
-    var_AddCallback(p_playlist, "intf-change", PlaylistChanged, p_intf);
-    var_AddCallback(p_playlist, "playlist-item-append", PlaylistChanged, p_intf);
-
-    while (vlc_object_alive(p_intf))
-    {
-        msleep(INTF_IDLE_SLEEP);
-
-        if (!p_sys->p_input)
-        {
-            p_sys->p_input = playlist_CurrentInput(p_playlist);
-            if (p_sys->p_input)
-                Redraw(p_intf, &t_last_refresh);
-        }
-        else if (p_sys->p_input->b_dead)
-        {
-            vlc_object_release(p_sys->p_input);
-            p_sys->p_input = NULL;
-        }
-
-        while (HandleKey(p_intf))
-            Redraw(p_intf, &t_last_refresh);
-
-        if ((time(0) - t_last_refresh) >= 1)
-            Redraw(p_intf, &t_last_refresh);
+    intf_thread_t *intf = data;
+    intf_sys_t    *sys = intf->p_sys;
+    playlist_t    *p_playlist = pl_Get(intf);
+
+    var_AddCallback(p_playlist, "intf-change", PlaylistChanged, intf);
+    var_AddCallback(p_playlist, "item-change", ItemChanged, intf);
+    var_AddCallback(p_playlist, "playlist-item-append", PlaylistChanged, intf);
+
+    vlc_cleanup_push(cleanup_run, data);
+    while (!sys->exit) {
+        UpdateInput(sys, p_playlist);
+        Redraw(intf);
+        HandleKey(intf);
     }
-    var_DelCallback(p_playlist, "intf-change", PlaylistChanged, p_intf);
-    var_DelCallback(p_playlist, "playlist-item-append", PlaylistChanged, p_intf);
-    vlc_restorecancel(canc);
+    vlc_cleanup_pop();
+
+    return NULL;
 }
 
 /*****************************************************************************
@@ -1815,58 +1834,52 @@ static void Run(intf_thread_t *p_intf)
  *****************************************************************************/
 static int Open(vlc_object_t *p_this)
 {
-    intf_thread_t *p_intf = (intf_thread_t *)p_this;
-    intf_sys_t    *p_sys  = p_intf->p_sys = calloc(1, sizeof(intf_sys_t));
-    struct msg_cb_data_t *msg_cb_data;
+    intf_thread_t *intf = (intf_thread_t *)p_this;
+    intf_sys_t    *sys  = intf->p_sys = calloc(1, sizeof(intf_sys_t));
+    playlist_t    *p_playlist = pl_Get(intf);
 
-    if (!p_sys)
+    if (!sys)
         return VLC_ENOMEM;
 
-    msg_cb_data = malloc(sizeof *msg_cb_data);
-    if (!msg_cb_data)
-    {
-        free(p_sys);
-        return VLC_ENOMEM;
-    }
+    vlc_mutex_init(&sys->msg_lock);
+    vlc_mutex_init(&sys->pl_lock);
 
-    msg_cb_data->p_sys = p_sys;
-    vlc_mutex_init(&p_sys->msg_lock);
-    p_sys->i_msgs = 0;
-    memset(p_sys->msgs, 0, sizeof p_sys->msgs);
-    p_sys->p_sub = msg_Subscribe(p_intf->p_libvlc, MsgCallback, msg_cb_data);
+    sys->verbosity = var_InheritInteger(intf, "verbose");
+    vlc_LogSet(intf->p_libvlc, MsgCallback, sys);
 
-    p_sys->i_box_type = BOX_PLAYLIST;
-    p_sys->b_plidx_follow = true;
-    p_sys->b_color = var_CreateGetBool(p_intf, "color");
+    sys->box_type = BOX_PLAYLIST;
+    sys->plidx_follow = true;
+    sys->color = var_CreateGetBool(intf, "color");
 
-    p_sys->psz_current_dir = var_CreateGetString(p_intf, "browse-dir");
-    if (!p_sys->psz_current_dir || !*p_sys->psz_current_dir)
-    {
-        free(p_sys->psz_current_dir);
-        p_sys->psz_current_dir = config_GetUserDir(VLC_HOME_DIR);
-    }
+    sys->current_dir = var_CreateGetNonEmptyString(intf, "browse-dir");
+    if (!sys->current_dir)
+        sys->current_dir = config_GetUserDir(VLC_HOME_DIR);
 
     initscr();   /* Initialize the curses library */
 
-    if (p_sys->b_color)
-        start_color_and_pairs(p_intf);
+    if (sys->color)
+        start_color_and_pairs(intf);
 
-    keypad(stdscr, TRUE);   /* Don't do NL -> CR/NL */
-    nonl();                 /* Take input chars one at a time */
-    cbreak();               /* Don't echo */
-    noecho();               /* Invisible cursor */
-    curs_set(0);            /* Non blocking wgetch() */
-    timeout(0);
+    keypad(stdscr, TRUE);
+    nonl();                 /* Don't do NL -> CR/NL */
+    cbreak();               /* Take input chars one at a time */
+    noecho();               /* Don't echo */
+    curs_set(0);            /* Invisible cursor */
+    timeout(1000);          /* blocking getch() */
     clear();
 
     /* Stop printing errors to the console */
-    if(!freopen("/dev/null", "wb", stderr))
-        msg_Err(p_intf, "Couldn't close stderr (%m)");
+    if (!freopen("/dev/null", "wb", stderr))
+        msg_Err(intf, "Couldn't close stderr (%s)", vlc_strerror_c(errno));
 
-    ReadDir(p_intf);
-    PlaylistRebuild(p_intf),
+    ReadDir(intf);
+    PL_LOCK;
+    PlaylistRebuild(intf),
+    PL_UNLOCK;
+
+    if (vlc_clone(&sys->thread, Run, intf, VLC_THREAD_PRIORITY_LOW))
+        abort(); /* TODO */
 
-    p_intf->pf_run = Run;
     return VLC_SUCCESS;
 }
 
@@ -1875,24 +1888,32 @@ static int Open(vlc_object_t *p_this)
  *****************************************************************************/
 static void Close(vlc_object_t *p_this)
 {
-    intf_sys_t *p_sys = ((intf_thread_t*)p_this)->p_sys;
+    intf_sys_t *sys = ((intf_thread_t*)p_this)->p_sys;
 
-    PlaylistDestroy(p_sys);
-    DirsDestroy(p_sys);
+    vlc_cancel(sys->thread);
+    vlc_join(sys->thread, NULL);
 
-    free(p_sys->psz_current_dir);
-    free(p_sys->psz_old_search);
+    PlaylistDestroy(sys);
+    DirsDestroy(sys);
 
-    if (p_sys->p_input)
-        vlc_object_release(p_sys->p_input);
+    free(sys->current_dir);
 
-    endwin();   /* Close the ncurses interface */
+    if (sys->p_input)
+        vlc_object_release(sys->p_input);
 
-    msg_Unsubscribe(p_sys->p_sub);
-    vlc_mutex_destroy(&p_sys->msg_lock);
-    for(unsigned i = 0; i < sizeof p_sys->msgs / sizeof *p_sys->msgs; i++)
-        if (p_sys->msgs[i])
-            msg_Release(p_sys->msgs[i]);
+    if (can_change_color())
+        /* Restore yellow to its original color */
+        init_color(COLOR_YELLOW, sys->yellow_r, sys->yellow_g, sys->yellow_b);
 
-    free(p_sys);
+    endwin();   /* Close the ncurses interface */
+
+    vlc_LogSet(p_this->p_libvlc, NULL, NULL);
+    vlc_mutex_destroy(&sys->msg_lock);
+    vlc_mutex_destroy(&sys->pl_lock);
+    for(unsigned i = 0; i < sizeof sys->msgs / sizeof *sys->msgs; i++) {
+        if (sys->msgs[i].item)
+            msg_Free(sys->msgs[i].item);
+        free(sys->msgs[i].msg);
+    }
+    free(sys);
 }