jpeg pushed a commit to branch master.

http://git.enlightenment.org/core/efl.git/commit/?id=1d99c82381043d9bf2670e5261bdf157d478b75b

commit 1d99c82381043d9bf2670e5261bdf157d478b75b
Author: Jean-Philippe Andre <[email protected]>
Date:   Mon Jan 13 17:15:20 2014 +0900

    Evas/cserve2: Improve error messages a bit
---
 src/bin/evas/evas_cserve2_cache.c      | 17 +++++++++--------
 src/bin/evas/evas_cserve2_main.c       |  4 ++--
 src/lib/evas/cserve2/evas_cs2_client.c |  9 ++++++++-
 3 files changed, 19 insertions(+), 11 deletions(-)

diff --git a/src/bin/evas/evas_cserve2_cache.c 
b/src/bin/evas/evas_cserve2_cache.c
index f4ce05e..d9f7064 100644
--- a/src/bin/evas/evas_cserve2_cache.c
+++ b/src/bin/evas/evas_cserve2_cache.c
@@ -1591,17 +1591,15 @@ _image_entry_new(Client *client, int rid,
    ref = eina_hash_find(client->files.referencing, &client_file_id);
    if (!ref)
      {
-        ERR("Couldn't find file id for client image id: %d",
-            client_file_id);
-        cserve2_client_error_send(client, rid,
-                                  CSERVE2_INVALID_CACHE);
+        ERR("Couldn't find file id for client image id: %d", client_file_id);
+        cserve2_client_error_send(client, rid, CSERVE2_INVALID_CACHE);
         return NULL;
      }
    fd = _file_data_find(ref->entry->id);
    if (!fd || fd->invalid)
      {
-        cserve2_client_error_send(client, rid,
-                                  CSERVE2_FILE_CHANGED);
+        ERR("Can't find file data %d (rid %d)", ref->entry->id, rid);
+        cserve2_client_error_send(client, rid, CSERVE2_FILE_CHANGED);
         return NULL;
      }
 
@@ -2922,8 +2920,8 @@ cserve2_cache_image_load(Client *client, unsigned int 
client_image_id, unsigned
    ref = eina_hash_find(client->images.referencing, &client_image_id);
    if (!ref)
      {
-        ERR("Can't load: client %d has no image id %d",
-            client->id, client_image_id);
+        ERR("Can't load: client %d has no image id %d (rid %d)",
+            client->id, client_image_id, rid);
         cserve2_client_error_send(client, rid, CSERVE2_NOT_LOADED);
         return;
      }
@@ -2932,6 +2930,7 @@ cserve2_cache_image_load(Client *client, unsigned int 
client_image_id, unsigned
    idata = _image_data_find(ENTRYID(ientry));
    if (!idata)
      {
+        ERR("Can't load image %d for rid %d: data not found", ENTRYID(ientry), 
rid);
         cserve2_client_error_send(client, rid, CSERVE2_INVALID_CACHE);
         return;
      }
@@ -2939,6 +2938,8 @@ cserve2_cache_image_load(Client *client, unsigned int 
client_image_id, unsigned
    fd = _file_data_find(idata->file_id);
    if (!fd || fd->invalid)
      {
+        ERR("Can't load image %d for rid %d%s", idata->file_id, rid,
+            fd->invalid ? ": invalid" : "");
         cserve2_client_error_send(client, rid, CSERVE2_FILE_CHANGED);
         return;
      }
diff --git a/src/bin/evas/evas_cserve2_main.c b/src/bin/evas/evas_cserve2_main.c
index 1e8fd18..1f2102a 100644
--- a/src/bin/evas/evas_cserve2_main.c
+++ b/src/bin/evas/evas_cserve2_main.c
@@ -144,8 +144,8 @@ _cserve2_client_open(Client *client)
    end = key + strlen(key) + 1;
 
    INF("Received OPEN command: RID=%d", msg->base.rid);
-   INF("File_ID: %d, path=\"%s\", key=\"%s\", has_load_opts=%d",
-       msg->file_id, path, key, (int) msg->has_load_opts);
+   INF("File_ID: %d, Image_ID: %d, path=\"%s\", key=\"%s\", has_load_opts=%d",
+       msg->file_id, msg->image_id, path, key, (int) msg->has_load_opts);
 
    if (!key[0]) key = NULL;
    if (msg->has_load_opts)
diff --git a/src/lib/evas/cserve2/evas_cs2_client.c 
b/src/lib/evas/cserve2/evas_cs2_client.c
index 15328f5..de1ce38 100644
--- a/src/lib/evas/cserve2/evas_cs2_client.c
+++ b/src/lib/evas/cserve2/evas_cs2_client.c
@@ -604,7 +604,14 @@ _server_dispatch(Eina_Bool *failed)
 
    rid = msg->rid;
    if (!found)
-     WRN("Got unexpected response %d for request %d", msg->type, rid);
+     {
+        if (msg->type == CSERVE2_ERROR)
+          {
+             Msg_Error *error = (Msg_Error *) msg;
+             ERR("Cserve2 sent error %d for rid %d", error->error, rid);
+          }
+        else WRN("Got unexpected response %d for request %d", msg->type, rid);
+     }
 
    free(msg);
    return rid;

-- 


Reply via email to