Code

fix typo in error message
[ncmpc.git] / src / mpdclient.c
index 24ac897527270cfc41cd425d3ef76e3919f49004..9baf70bc7134fa8d2125f4ff463660052eb3ff82 100644 (file)
@@ -1,21 +1,21 @@
 /* ncmpc (Ncurses MPD Client)
  * (c) 2004-2010 The Music Player Daemon Project
  * Project homepage: http://musicpd.org
-
+ *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
  * the Free Software Foundation; either version 2 of the License, or
  * (at your option) any later version.
-
+ *
  * This program is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
  * GNU General Public License for more details.
-
+ *
  * You should have received a copy of the GNU General Public License along
  * with this program; if not, write to the Free Software Foundation, Inc.,
  * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
-*/
+ */
 
 #include "mpdclient.h"
 #include "filelist.h"
@@ -40,16 +40,16 @@ gint
 compare_filelistentry_format(gconstpointer filelist_entry1,
                             gconstpointer filelist_entry2)
 {
-       const struct mpd_entity *e1, *e2;
-       char key1[BUFSIZE], key2[BUFSIZE];
-       int n = 0;
-
-       e1 = ((const struct filelist_entry *)filelist_entry1)->entity;
-       e2 = ((const struct filelist_entry *)filelist_entry2)->entity;
+       const struct mpd_entity *e1 =
+               ((const struct filelist_entry *)filelist_entry1)->entity;
+       const struct mpd_entity *e2 =
+               ((const struct filelist_entry *)filelist_entry2)->entity;
 
+       int n = 0;
        if (e1 && e2 &&
            mpd_entity_get_type(e1) == MPD_ENTITY_TYPE_SONG &&
            mpd_entity_get_type(e2) == MPD_ENTITY_TYPE_SONG) {
+               char key1[BUFSIZE], key2[BUFSIZE];
                strfsong(key1, BUFSIZE, options.list_format, mpd_entity_get_song(e1));
                strfsong(key2, BUFSIZE, options.list_format, mpd_entity_get_song(e2));
                n = strcmp(key1,key2);
@@ -83,19 +83,10 @@ mpdclient_handle_error(struct mpdclient *c)
        return false;
 }
 
-static bool
-mpdclient_finish_command(struct mpdclient *c)
-{
-       return mpd_response_finish(c->connection)
-               ? true : mpdclient_handle_error(c);
-}
-
 struct mpdclient *
 mpdclient_new(void)
 {
-       struct mpdclient *c;
-
-       c = g_new0(struct mpdclient, 1);
+       struct mpdclient *c = g_new0(struct mpdclient, 1);
        playlist_init(&c->playlist);
        c->volume = -1;
        c->events = 0;
@@ -122,8 +113,10 @@ mpdclient_disconnect(struct mpdclient *c)
                c->idle = false;
        }
 
-       if (c->connection)
+       if (c->connection) {
                mpd_connection_free(c->connection);
+               ++c->connection_id;
+       }
        c->connection = NULL;
 
        if (c->status)
@@ -136,9 +129,7 @@ mpdclient_disconnect(struct mpdclient *c)
                c->song = NULL;
 
        /* everything has changed after a disconnect */
-       c->events |= MPD_IDLE_DATABASE|MPD_IDLE_STORED_PLAYLIST|
-               MPD_IDLE_QUEUE|MPD_IDLE_PLAYER|MPD_IDLE_MIXER|MPD_IDLE_OUTPUT|
-               MPD_IDLE_OPTIONS|MPD_IDLE_UPDATE;
+       c->events |= MPD_IDLE_ALL;
 }
 
 bool
@@ -170,6 +161,8 @@ mpdclient_connect(struct mpdclient *c,
                return false;
        }
 
+       ++c->connection_id;
+
        return true;
 }
 
@@ -261,11 +254,9 @@ mpdclient_put_connection(struct mpdclient *c)
 static struct mpd_status *
 mpdclient_recv_status(struct mpdclient *c)
 {
-       struct mpd_status *status;
-
        assert(c->connection != NULL);
 
-       status = mpd_recv_status(c->connection);
+       struct mpd_status *status = mpd_recv_status(c->connection);
        if (status == NULL) {
                mpdclient_handle_error(c);
                return NULL;
@@ -283,32 +274,24 @@ mpdclient_recv_status(struct mpdclient *c)
 bool
 mpdclient_cmd_crop(struct mpdclient *c)
 {
-       struct mpd_connection *connection;
-       int length, current;
-
        if (!mpdclient_is_playing(c))
                return false;
 
-       length = mpd_status_get_queue_length(c->status);
-       current = mpd_status_get_song_pos(c->status);
+       int length = mpd_status_get_queue_length(c->status);
+       int current = mpd_status_get_song_pos(c->status);
        if (current < 0 || mpd_status_get_queue_length(c->status) < 2)
                return true;
 
-       connection = mpdclient_get_connection(c);
+       struct mpd_connection *connection = mpdclient_get_connection(c);
        if (connection == NULL)
                return false;
 
        mpd_command_list_begin(connection, false);
 
-       if (mpd_connection_cmp_server_version(connection, 0, 16, 0) >= 0) {
-               if (current < length - 1)
-                       mpd_send_delete_range(connection, current + 1, length);
-               if (current > 0)
-                       mpd_send_delete_range(connection, 0, current);
-       } else
-               while (--length >= 0)
-                       if (length != current)
-                               mpd_send_delete(connection, length);
+       if (current < length - 1)
+               mpd_send_delete_range(connection, current + 1, length);
+       if (current > 0)
+               mpd_send_delete_range(connection, 0, current);
 
        mpd_command_list_end(connection);
 
@@ -319,8 +302,6 @@ bool
 mpdclient_cmd_clear(struct mpdclient *c)
 {
        struct mpd_connection *connection = mpdclient_get_connection(c);
-       struct mpd_status *status;
-
        if (connection == NULL)
                return false;
 
@@ -333,7 +314,7 @@ mpdclient_cmd_clear(struct mpdclient *c)
 
        /* receive the new status, store it in the mpdclient struct */
 
-       status = mpdclient_recv_status(c);
+       struct mpd_status *status = mpdclient_recv_status(c);
        if (status == NULL)
                return false;
 
@@ -412,20 +393,17 @@ mpdclient_cmd_add_path(struct mpdclient *c, const gchar *path_utf8)
        if (connection == NULL)
                return false;
 
-       mpd_send_add(connection, path_utf8);
-       return mpdclient_finish_command(c);
+       return mpd_send_add(connection, path_utf8)?
+               mpdclient_finish_command(c) : false;
 }
 
 bool
 mpdclient_cmd_add(struct mpdclient *c, const struct mpd_song *song)
 {
-       struct mpd_connection *connection = mpdclient_get_connection(c);
-       struct mpd_status *status;
-       struct mpd_song *new_song;
-
        assert(c != NULL);
        assert(song != NULL);
 
+       struct mpd_connection *connection = mpdclient_get_connection(c);
        if (connection == NULL || c->status == NULL)
                return false;
 
@@ -444,14 +422,14 @@ mpdclient_cmd_add(struct mpdclient *c, const struct mpd_song *song)
 
        c->events |= MPD_IDLE_QUEUE;
 
-       status = mpdclient_recv_status(c);
+       struct mpd_status *status = mpdclient_recv_status(c);
        if (status == NULL)
                return false;
 
        if (!mpd_response_next(connection))
                return mpdclient_handle_error(c);
 
-       new_song = mpd_recv_song(connection);
+       struct mpd_song *new_song = mpd_recv_song(connection);
        if (!mpd_response_finish(connection) || new_song == NULL) {
                if (new_song != NULL)
                        mpd_song_free(new_song);
@@ -481,8 +459,6 @@ bool
 mpdclient_cmd_delete(struct mpdclient *c, gint idx)
 {
        struct mpd_connection *connection = mpdclient_get_connection(c);
-       const struct mpd_song *song;
-       struct mpd_status *status;
 
        if (connection == NULL || c->status == NULL)
                return false;
@@ -490,7 +466,7 @@ mpdclient_cmd_delete(struct mpdclient *c, gint idx)
        if (idx < 0 || (guint)idx >= playlist_length(&c->playlist))
                return false;
 
-       song = playlist_get(&c->playlist, idx);
+       const struct mpd_song *song = playlist_get(&c->playlist, idx);
 
        /* send the delete command to mpd; at the same time, get the
           new status (to verify the playlist id) */
@@ -503,7 +479,7 @@ mpdclient_cmd_delete(struct mpdclient *c, gint idx)
 
        c->events |= MPD_IDLE_QUEUE;
 
-       status = mpdclient_recv_status(c);
+       struct mpd_status *status = mpdclient_recv_status(c);
        if (status == NULL)
                return false;
 
@@ -528,52 +504,18 @@ mpdclient_cmd_delete(struct mpdclient *c, gint idx)
        return true;
 }
 
-/**
- * Fallback for mpdclient_cmd_delete_range() on MPD older than 0.16.
- * It emulates the "delete range" command with a list of simple
- * "delete" commands.
- */
-static bool
-mpdclient_cmd_delete_range_fallback(struct mpdclient *c,
-                                   unsigned start, unsigned end)
-{
-       struct mpd_connection *connection = mpdclient_get_connection(c);
-       if (connection == NULL)
-               return false;
-
-       if (!mpd_command_list_begin(connection, false))
-               return mpdclient_handle_error(c);
-
-       for (; start < end; --end)
-               mpd_send_delete(connection, start);
-
-       if (!mpd_command_list_end(connection) ||
-           !mpd_response_finish(connection))
-               return mpdclient_handle_error(c);
-
-       return true;
-}
-
 bool
 mpdclient_cmd_delete_range(struct mpdclient *c, unsigned start, unsigned end)
 {
-       struct mpd_connection *connection;
-       struct mpd_status *status;
-
        if (end == start + 1)
                /* if that's not really a range, we choose to use the
                   safer "deleteid" version */
                return mpdclient_cmd_delete(c, start);
 
-       connection = mpdclient_get_connection(c);
+       struct mpd_connection *connection = mpdclient_get_connection(c);
        if (connection == NULL)
                return false;
 
-       if (mpd_connection_cmp_server_version(connection, 0, 16, 0) < 0)
-               return mpdclient_cmd_delete_range_fallback(c, start, end);
-
-       /* MPD 0.16 supports "delete" with a range argument */
-
        /* send the delete command to mpd; at the same time, get the
           new status (to verify the playlist id) */
 
@@ -585,7 +527,7 @@ mpdclient_cmd_delete_range(struct mpdclient *c, unsigned start, unsigned end)
 
        c->events |= MPD_IDLE_QUEUE;
 
-       status = mpdclient_recv_status(c);
+       struct mpd_status *status = mpdclient_recv_status(c);
        if (status == NULL)
                return false;
 
@@ -617,13 +559,10 @@ mpdclient_cmd_delete_range(struct mpdclient *c, unsigned start, unsigned end)
 bool
 mpdclient_cmd_move(struct mpdclient *c, unsigned dest_pos, unsigned src_pos)
 {
-       struct mpd_connection *connection;
-       struct mpd_status *status;
-
        if (dest_pos == src_pos)
                return true;
 
-       connection = mpdclient_get_connection(c);
+       struct mpd_connection *connection = mpdclient_get_connection(c);
        if (connection == NULL)
                return false;
 
@@ -638,7 +577,7 @@ mpdclient_cmd_move(struct mpdclient *c, unsigned dest_pos, unsigned src_pos)
 
        c->events |= MPD_IDLE_QUEUE;
 
-       status = mpdclient_recv_status(c);
+       struct mpd_status *status = mpdclient_recv_status(c);
        if (status == NULL)
                return false;
 
@@ -659,6 +598,76 @@ mpdclient_cmd_move(struct mpdclient *c, unsigned dest_pos, unsigned src_pos)
        return true;
 }
 
+#if LIBMPDCLIENT_CHECK_VERSION(2,5,0)
+/* The client-to-client protocol (MPD 0.17.0) */
+
+bool
+mpdclient_cmd_subscribe(struct mpdclient *c, const char *channel)
+{
+       struct mpd_connection *connection = mpdclient_get_connection(c);
+
+       if (connection == NULL)
+               return false;
+
+       if (!mpd_send_subscribe(connection, channel))
+               return mpdclient_handle_error(c);
+
+       return mpdclient_finish_command(c);
+}
+
+bool
+mpdclient_cmd_unsubscribe(struct mpdclient *c, const char *channel)
+{
+       struct mpd_connection *connection = mpdclient_get_connection(c);
+       if (connection == NULL)
+               return false;
+
+       if (!mpd_send_unsubscribe(connection, channel))
+               return mpdclient_handle_error(c);
+
+       return mpdclient_finish_command(c);
+}
+
+bool
+mpdclient_cmd_send_message(struct mpdclient *c, const char *channel,
+                          const char *text)
+{
+       struct mpd_connection *connection = mpdclient_get_connection(c);
+       if (connection == NULL)
+               return false;
+
+       if (!mpd_send_send_message(connection, channel, text))
+               return mpdclient_handle_error(c);
+
+       return mpdclient_finish_command(c);
+}
+
+bool
+mpdclient_send_read_messages(struct mpdclient *c)
+{
+       struct mpd_connection *connection = mpdclient_get_connection(c);
+       if (connection == NULL)
+               return false;
+
+       return mpd_send_read_messages(connection)?
+               true : mpdclient_handle_error(c);
+}
+
+struct mpd_message *
+mpdclient_recv_message(struct mpdclient *c)
+{
+       struct mpd_connection *connection = mpdclient_get_connection(c);
+       if (connection == NULL)
+               return false;
+
+       struct mpd_message *message = mpd_recv_message(connection);
+       if (message == NULL &&
+           mpd_connection_get_error(connection) != MPD_ERROR_SUCCESS)
+               mpdclient_handle_error(c);
+
+       return message;
+}
+#endif
 
 /****************************************************************************/
 /*** Playlist management functions ******************************************/
@@ -669,14 +678,14 @@ bool
 mpdclient_playlist_update(struct mpdclient *c)
 {
        struct mpd_connection *connection = mpdclient_get_connection(c);
-       struct mpd_entity *entity;
-
        if (connection == NULL)
                return false;
 
        playlist_clear(&c->playlist);
 
        mpd_send_list_queue_meta(connection);
+
+       struct mpd_entity *entity;
        while ((entity = mpd_recv_entity(connection))) {
                if (mpd_entity_get_type(entity) == MPD_ENTITY_TYPE_SONG)
                        playlist_append(&c->playlist, mpd_entity_get_song(entity));
@@ -695,14 +704,13 @@ bool
 mpdclient_playlist_update_changes(struct mpdclient *c)
 {
        struct mpd_connection *connection = mpdclient_get_connection(c);
-       struct mpd_song *song;
-       guint length;
 
        if (connection == NULL)
                return false;
 
        mpd_send_queue_changes_meta(connection, c->playlist.version);
 
+       struct mpd_song *song;
        while ((song = mpd_recv_song(connection)) != NULL) {
                int pos = mpd_song_get_pos(song);
 
@@ -719,7 +727,7 @@ mpdclient_playlist_update_changes(struct mpdclient *c)
 
        /* remove trailing songs */
 
-       length = mpd_status_get_queue_length(c->status);
+       unsigned length = mpd_status_get_queue_length(c->status);
        while (length < c->playlist.list->len) {
                guint pos = c->playlist.list->len - 1;
 
@@ -742,8 +750,6 @@ bool
 mpdclient_filelist_add_all(struct mpdclient *c, struct filelist *fl)
 {
        struct mpd_connection *connection = mpdclient_get_connection(c);
-       guint i;
-
        if (connection == NULL)
                return false;
 
@@ -752,7 +758,7 @@ mpdclient_filelist_add_all(struct mpdclient *c, struct filelist *fl)
 
        mpd_command_list_begin(connection, false);
 
-       for (i = 0; i < filelist_length(fl); ++i) {
+       for (unsigned i = 0; i < filelist_length(fl); ++i) {
                struct filelist_entry *entry = filelist_get(fl, i);
                struct mpd_entity *entity  = entry->entity;