Make all search results visible.

[disorder] / lib / eclient.c

diff --git a/lib/eclient.c b/lib/eclient.c
index a753a52de8bf40c40f3cdfb2fb6d9e28d17afd5a..69069f5a7d0683420db2f597a2025999412321e3 100644 (file)
--- a/lib/eclient.c
+++ b/lib/eclient.c
@@ -913,7 +913,10 @@ static void eclient_queue_error(const char *msg,
 /* for commands that expect a queue dump */
 static void queue_response_opcallback(disorder_eclient *c,
                                       struct operation *op) {
 /* for commands that expect a queue dump */
 static void queue_response_opcallback(disorder_eclient *c,
                                       struct operation *op) {

+  disorder_eclient_queue_response *const completed
+    = (disorder_eclient_queue_response *)op->completed;

   int n;
   int n;

+  int parse_failed = 0;

   struct queue_entry *q, *qh = 0, **qtail = &qh, *qlast = 0;
   
   D(("queue_response_callback"));
   struct queue_entry *q, *qh = 0, **qtail = &qh, *qlast = 0;
   
   D(("queue_response_callback"));


@@ -922,23 +925,29 @@ static void queue_response_opcallback(disorder_eclient *c,
     for(n = 0; n < c->vec.nvec; ++n) {
       q = xmalloc(sizeof *q);
       D(("queue_unmarshall %s", c->vec.vec[n]));
     for(n = 0; n < c->vec.nvec; ++n) {
       q = xmalloc(sizeof *q);
       D(("queue_unmarshall %s", c->vec.vec[n]));

-      if(!queue_unmarshall(q, c->vec.vec[n], eclient_queue_error, op)) {
+      if(!queue_unmarshall(q, c->vec.vec[n], NULL, op)) {

         q->prev = qlast;
         *qtail = q;
         qtail = &q->next;
         qlast = q;
         q->prev = qlast;
         *qtail = q;
         qtail = &q->next;
         qlast = q;

-      }
+      } else
+        parse_failed = 1;

     }
     }

-    if(op->completed)
-      ((disorder_eclient_queue_response *)op->completed)(op->v, qh);
+    /* Currently we pass the partial queue to the callback along with the
+     * error.  This might not be very useful in practice... */
+    if(parse_failed)
+      completed(op->v, "cannot parse result", qh);
+    else
+      completed(op->v, 0, qh);

   } else
   } else

-    /* TODO don't use protocol_error here */
-    protocol_error(c, op, c->rc, "%s: %s", c->ident, c->line);
+    completed(op->v, errorstring(c), 0);

 } 
 
 /* for 'playing' */
 static void playing_response_opcallback(disorder_eclient *c,
                                         struct operation *op) {
 } 
 
 /* for 'playing' */
 static void playing_response_opcallback(disorder_eclient *c,
                                         struct operation *op) {

+  disorder_eclient_queue_response *const completed
+    = (disorder_eclient_queue_response *)op->completed;

   struct queue_entry *q;
 
   D(("playing_response_callback"));
   struct queue_entry *q;
 
   D(("playing_response_callback"));


@@ -946,36 +955,33 @@ static void playing_response_opcallback(disorder_eclient *c,
     switch(c->rc % 10) {
     case 2:
       if(queue_unmarshall(q = xmalloc(sizeof *q), c->line + 4,
     switch(c->rc % 10) {
     case 2:
       if(queue_unmarshall(q = xmalloc(sizeof *q), c->line + 4,

-                          eclient_queue_error, c))
-        return;
+                          NULL, c))
+        completed(op->v, "cannot parse result", 0);
+      else
+        completed(op->v, 0, q);

       break;
     case 9:
       break;
     case 9:

-      q = 0;
+      completed(op->v, 0, 0);

       break;
     default:
       break;
     default:

-      /* TODO don't use protocol_error here */
-      protocol_error(c, op, c->rc, "%s: %s", c->ident, c->line);
-      return;
+      completed(op->v, errorstring(c), 0);
+      break;

     }
     }

-    if(op->completed)
-      ((disorder_eclient_queue_response *)op->completed)(op->v, q);

   } else
   } else

-    /* TODO don't use protocol_error here */
-    protocol_error(c, op, c->rc, "%s: %s", c->ident, c->line);
+    completed(op->v, errorstring(c), 0);

 }
 
 /* for commands that expect a list of some sort */
 static void list_response_opcallback(disorder_eclient *c,
                                      struct operation *op) {
 }
 
 /* for commands that expect a list of some sort */
 static void list_response_opcallback(disorder_eclient *c,
                                      struct operation *op) {

+  disorder_eclient_list_response *const completed =
+    (disorder_eclient_list_response *)op->completed;
+

   D(("list_response_callback"));
   D(("list_response_callback"));

-  if(c->rc / 100 == 2) {
-    if(op->completed)
-      ((disorder_eclient_list_response *)op->completed)(op->v,
-                                                        c->vec.nvec,
-                                                        c->vec.vec);
-  } else
-    /* TODO don't use protocol_error here */
-    protocol_error(c, op, c->rc, "%s: %s", c->ident, c->line);
+  if(c->rc / 100 == 2)
+    completed(op->v, NULL, c->vec.nvec, c->vec.vec);
+  else
+    completed(op->v, errorstring(c), 0, 0);

 }
 
 /* for volume */
 }
 
 /* for volume */


@@ -1262,17 +1268,19 @@ int disorder_eclient_new_tracks(disorder_eclient *c,
 
 static void rtp_response_opcallback(disorder_eclient *c,
                                     struct operation *op) {
 
 static void rtp_response_opcallback(disorder_eclient *c,
                                     struct operation *op) {

+  disorder_eclient_list_response *const completed =
+    (disorder_eclient_list_response *)op->completed;

   D(("rtp_response_opcallback"));
   if(c->rc / 100 == 2) {
   D(("rtp_response_opcallback"));
   if(c->rc / 100 == 2) {

-    if(op->completed) {
-      int nvec;
-      char **vec = split(c->line + 4, &nvec, SPLIT_QUOTES, 0, 0);
+    int nvec;
+    char **vec = split(c->line + 4, &nvec, SPLIT_QUOTES, 0, 0);

 
 

-      ((disorder_eclient_list_response *)op->completed)(op->v, nvec, vec);
-    }
+    if(vec)
+      completed(op->v, NULL, nvec, vec);
+    else
+      completed(op->v, "error parsing response", 0, 0);

   } else
   } else

-    /* TODO don't use protocol_error here */
-    protocol_error(c, op, c->rc, "%s: %s", c->ident, c->line);
+    completed(op->v, errorstring(c), 0, 0);

 }
 
 /** @brief Determine the RTP target address
 }
 
 /** @brief Determine the RTP target address