cedric pushed a commit to branch master.

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

commit a299916d84aedae9929c24bce3b83007719f2f73
Author: wonguk.jeong <[email protected]>
Date:   Wed Jul 9 14:00:40 2014 +0200

    ethumb: dbus reply message handling minor bug fix ("queue_remove" method 
call)
    
    Summary:
    due to wrong return value checking, annoying error message is coming out.
    
    @fix
    
    Test Plan: terminology -> tyls in the directory contains pictures -> clear 
screen (clear) -> check whether error message is coming out or not
    
    Reviewers: raster, cedric
    
    CC: seoz, cedric
    
    Differential Revision: https://phab.enlightenment.org/D1159
    
    Signed-off-by: Cedric BAIL <[email protected]>
---
 src/lib/ethumb_client/ethumb_client.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/lib/ethumb_client/ethumb_client.c 
b/src/lib/ethumb_client/ethumb_client.c
index cec7e76..dbc9681 100644
--- a/src/lib/ethumb_client/ethumb_client.c
+++ b/src/lib/ethumb_client/ethumb_client.c
@@ -1040,7 +1040,7 @@ _ethumb_client_queue_remove_cb(void *data, const 
Eldbus_Message *msg, Eldbus_Pen
         goto end;
      }
 
-   if (eldbus_message_arguments_get(msg, "b", &success))
+   if (!eldbus_message_arguments_get(msg, "b", &success))
      {
         ERR("Error getting arguments.");
         goto end;

-- 


Reply via email to