diff options
Diffstat (limited to 'src')
-rw-r--r-- | src/wkb-ibus-config.c | 13 | ||||
-rw-r--r-- | src/wkb-ibus-panel.c | 55 | ||||
-rw-r--r-- | src/wkb-ibus.c | 12 |
3 files changed, 39 insertions, 41 deletions
diff --git a/src/wkb-ibus-config.c b/src/wkb-ibus-config.c index 5ed7535..9c7ea8b 100644 --- a/src/wkb-ibus-config.c +++ b/src/wkb-ibus-config.c @@ -18,6 +18,7 @@ #include <stdlib.h> #include <string.h> +#include <Eina.h> #include <Eldbus.h> #include "wkb-ibus.h" @@ -26,7 +27,7 @@ static struct wkb_ibus_config_eet *_conf_eet = NULL; -#define CONFIG_CHECK_MESSAGE_ERRORS(_msg) \ +#define _config_check_message_errors(_msg) \ do \ { \ const char *error, *error_msg; \ @@ -36,7 +37,7 @@ static struct wkb_ibus_config_eet *_conf_eet = NULL; return NULL; \ } \ DBG("Message '%s' with signature '%s'", eldbus_message_member_get(_msg), eldbus_message_signature_get(_msg)); \ - } while (0); + } while (0) static Eldbus_Message * _config_set_value(const Eldbus_Service_Interface *iface, const Eldbus_Message *msg) @@ -44,7 +45,7 @@ _config_set_value(const Eldbus_Service_Interface *iface, const Eldbus_Message *m const char *section, *name; Eldbus_Message_Iter *value; - CONFIG_CHECK_MESSAGE_ERRORS(msg) + _config_check_message_errors(msg); if (!eldbus_message_arguments_get(msg, "ssv", §ion, &name, &value)) { @@ -62,7 +63,7 @@ _config_get_value(const Eldbus_Service_Interface *iface, const Eldbus_Message *m { const char *section, *name; - CONFIG_CHECK_MESSAGE_ERRORS(msg) + _config_check_message_errors(msg); if (!eldbus_message_arguments_get(msg, "ss", §ion, &name)) { @@ -80,7 +81,7 @@ _config_get_values(const Eldbus_Service_Interface *iface, const Eldbus_Message * { const char *section; - CONFIG_CHECK_MESSAGE_ERRORS(msg) + _config_check_message_errors(msg); if (!eldbus_message_arguments_get(msg, "s", §ion)) { @@ -98,7 +99,7 @@ _config_unset_value(const Eldbus_Service_Interface *iface, const Eldbus_Message { const char *section, *name; - CONFIG_CHECK_MESSAGE_ERRORS(msg) + _config_check_message_errors(msg); if (!eldbus_message_arguments_get(msg, "ss", §ion, &name)) { diff --git a/src/wkb-ibus-panel.c b/src/wkb-ibus-panel.c index 9832c06..dfa77e9 100644 --- a/src/wkb-ibus-panel.c +++ b/src/wkb-ibus-panel.c @@ -18,12 +18,13 @@ #include <stdlib.h> #include <string.h> +#include <Eina.h> #include <Eldbus.h> #include "wkb-ibus.h" #include "wkb-ibus-defs.h" -#define PANEL_CHECK_MESSAGE_ERRORS(_msg) \ +#define _panel_check_message_errors(_msg) \ do \ { \ const char *error, *error_msg; \ @@ -33,7 +34,7 @@ return NULL; \ } \ DBG("Message '%s' with signature '%s'", eldbus_message_member_get(_msg), eldbus_message_signature_get(_msg)); \ - } while (0); + } while (0) static Eina_Array *_get_properties_from_message_iter(Eldbus_Message_Iter *iter); @@ -427,7 +428,7 @@ _panel_update_preedit_text(const Eldbus_Service_Interface *iface, const Eldbus_M Eina_Bool visible = 0; struct _ibus_text *ibus_text; - PANEL_CHECK_MESSAGE_ERRORS(msg) + _panel_check_message_errors(msg); if (!eldbus_message_arguments_get(msg, "vub", &text, &cursor_pos, &visible)) { @@ -447,7 +448,7 @@ _panel_update_preedit_text(const Eldbus_Service_Interface *iface, const Eldbus_M static Eldbus_Message * _panel_show_preedit_text(const Eldbus_Service_Interface *iface, const Eldbus_Message *msg) { - PANEL_CHECK_MESSAGE_ERRORS(msg) + _panel_check_message_errors(msg); return NULL; } @@ -455,7 +456,7 @@ _panel_show_preedit_text(const Eldbus_Service_Interface *iface, const Eldbus_Mes static Eldbus_Message * _panel_hide_preedit_text(const Eldbus_Service_Interface *iface, const Eldbus_Message *msg) { - PANEL_CHECK_MESSAGE_ERRORS(msg) + _panel_check_message_errors(msg); return NULL; } @@ -467,7 +468,7 @@ _panel_update_auxiliary_text(const Eldbus_Service_Interface *iface, const Eldbus Eina_Bool visible = 0; struct _ibus_text *ibus_text; - PANEL_CHECK_MESSAGE_ERRORS(msg) + _panel_check_message_errors(msg); if (!eldbus_message_arguments_get(msg, "vb", &text, &visible)) { @@ -487,7 +488,7 @@ _panel_update_auxiliary_text(const Eldbus_Service_Interface *iface, const Eldbus static Eldbus_Message * _panel_show_auxiliary_text(const Eldbus_Service_Interface *iface, const Eldbus_Message *msg) { - PANEL_CHECK_MESSAGE_ERRORS(msg) + _panel_check_message_errors(msg); return NULL; } @@ -495,7 +496,7 @@ _panel_show_auxiliary_text(const Eldbus_Service_Interface *iface, const Eldbus_M static Eldbus_Message * _panel_hide_auxiliary_text(const Eldbus_Service_Interface *iface, const Eldbus_Message *msg) { - PANEL_CHECK_MESSAGE_ERRORS(msg) + _panel_check_message_errors(msg); return NULL; } @@ -507,7 +508,7 @@ _panel_update_lookup_table(const Eldbus_Service_Interface *iface, const Eldbus_M Eina_Bool visible = 0; struct _ibus_lookup_table *ibus_lookup_table; - PANEL_CHECK_MESSAGE_ERRORS(msg) + _panel_check_message_errors(msg); if (!eldbus_message_arguments_get(msg, "vb", &table, &visible)) { @@ -526,7 +527,7 @@ _panel_update_lookup_table(const Eldbus_Service_Interface *iface, const Eldbus_M static Eldbus_Message * _panel_show_lookup_table(const Eldbus_Service_Interface *iface, const Eldbus_Message *msg) { - PANEL_CHECK_MESSAGE_ERRORS(msg) + _panel_check_message_errors(msg); return NULL; } @@ -534,9 +535,7 @@ _panel_show_lookup_table(const Eldbus_Service_Interface *iface, const Eldbus_Mes static Eldbus_Message * _panel_hide_lookup_table(const Eldbus_Service_Interface *iface, const Eldbus_Message *msg) { - PANEL_CHECK_MESSAGE_ERRORS(msg) - - DBG("here"); + _panel_check_message_errors(msg); return NULL; } @@ -544,9 +543,7 @@ _panel_hide_lookup_table(const Eldbus_Service_Interface *iface, const Eldbus_Mes static Eldbus_Message * _panel_cursor_up_lookup_table(const Eldbus_Service_Interface *iface, const Eldbus_Message *msg) { - PANEL_CHECK_MESSAGE_ERRORS(msg) - - DBG("here"); + _panel_check_message_errors(msg); return NULL; } @@ -554,7 +551,7 @@ _panel_cursor_up_lookup_table(const Eldbus_Service_Interface *iface, const Eldbu static Eldbus_Message * _panel_cursor_down_lookup_table(const Eldbus_Service_Interface *iface, const Eldbus_Message *msg) { - PANEL_CHECK_MESSAGE_ERRORS(msg) + _panel_check_message_errors(msg); return NULL; } @@ -562,7 +559,7 @@ _panel_cursor_down_lookup_table(const Eldbus_Service_Interface *iface, const Eld static Eldbus_Message * _panel_page_up_lookup_table(const Eldbus_Service_Interface *iface, const Eldbus_Message *msg) { - PANEL_CHECK_MESSAGE_ERRORS(msg) + _panel_check_message_errors(msg); return NULL; } @@ -570,7 +567,7 @@ _panel_page_up_lookup_table(const Eldbus_Service_Interface *iface, const Eldbus_ static Eldbus_Message * _panel_page_down_lookup_table(const Eldbus_Service_Interface *iface, const Eldbus_Message *msg) { - PANEL_CHECK_MESSAGE_ERRORS(msg) + _panel_check_message_errors(msg); return NULL; } @@ -581,7 +578,7 @@ _panel_register_properties(const Eldbus_Service_Interface *iface, const Eldbus_M Eldbus_Message_Iter *props = NULL; Eina_Array *properties = NULL; - PANEL_CHECK_MESSAGE_ERRORS(msg) + _panel_check_message_errors(msg); if (!eldbus_message_arguments_get(msg, "v", &props)) { @@ -602,7 +599,7 @@ _panel_update_property(const Eldbus_Service_Interface *iface, const Eldbus_Messa { Eldbus_Message_Iter *prop = NULL; - PANEL_CHECK_MESSAGE_ERRORS(msg) + _panel_check_message_errors(msg); if (!eldbus_message_arguments_get(msg, "v", &prop)) { @@ -619,7 +616,7 @@ _panel_update_property(const Eldbus_Service_Interface *iface, const Eldbus_Messa static Eldbus_Message * _panel_focus_in(const Eldbus_Service_Interface *iface, const Eldbus_Message *msg) { - PANEL_CHECK_MESSAGE_ERRORS(msg) + _panel_check_message_errors(msg); return NULL; } @@ -627,7 +624,7 @@ _panel_focus_in(const Eldbus_Service_Interface *iface, const Eldbus_Message *msg static Eldbus_Message * _panel_focus_out(const Eldbus_Service_Interface *iface, const Eldbus_Message *msg) { - PANEL_CHECK_MESSAGE_ERRORS(msg) + _panel_check_message_errors(msg); return NULL; } @@ -637,7 +634,7 @@ _panel_set_cursor_location(const Eldbus_Service_Interface *iface, const Eldbus_M { int x = 0, y = 0, w = 0, h = 0; - PANEL_CHECK_MESSAGE_ERRORS(msg) + _panel_check_message_errors(msg); if (!eldbus_message_arguments_get(msg, "iiii", &x, &y, &w, &h)) { @@ -653,7 +650,7 @@ _panel_set_cursor_location(const Eldbus_Service_Interface *iface, const Eldbus_M static Eldbus_Message * _panel_reset(const Eldbus_Service_Interface *iface, const Eldbus_Message *msg) { - PANEL_CHECK_MESSAGE_ERRORS(msg) + _panel_check_message_errors(msg); return NULL; } @@ -661,7 +658,7 @@ _panel_reset(const Eldbus_Service_Interface *iface, const Eldbus_Message *msg) static Eldbus_Message * _panel_start_setup(const Eldbus_Service_Interface *iface, const Eldbus_Message *msg) { - PANEL_CHECK_MESSAGE_ERRORS(msg) + _panel_check_message_errors(msg); return NULL; } @@ -669,7 +666,7 @@ _panel_start_setup(const Eldbus_Service_Interface *iface, const Eldbus_Message * static Eldbus_Message * _panel_state_changed(const Eldbus_Service_Interface *iface, const Eldbus_Message *msg) { - PANEL_CHECK_MESSAGE_ERRORS(msg) + _panel_check_message_errors(msg); return NULL; } @@ -677,7 +674,7 @@ _panel_state_changed(const Eldbus_Service_Interface *iface, const Eldbus_Message static Eldbus_Message * _panel_hide_language_bar(const Eldbus_Service_Interface *iface, const Eldbus_Message *msg) { - PANEL_CHECK_MESSAGE_ERRORS(msg) + _panel_check_message_errors(msg); return NULL; } @@ -685,7 +682,7 @@ _panel_hide_language_bar(const Eldbus_Service_Interface *iface, const Eldbus_Mes static Eldbus_Message * _panel_show_language_bar(const Eldbus_Service_Interface *iface, const Eldbus_Message *msg) { - PANEL_CHECK_MESSAGE_ERRORS(msg) + _panel_check_message_errors(msg); return NULL; } diff --git a/src/wkb-ibus.c b/src/wkb-ibus.c index 882a9dd..55e4e26 100644 --- a/src/wkb-ibus.c +++ b/src/wkb-ibus.c @@ -27,7 +27,7 @@ int _wkb_ibus_log_dom = -1; -#define CHECK_MESSAGE_ERRORS(_msg) \ +#define _check_message_errors(_msg) \ do \ { \ const char *error, *error_msg; \ @@ -37,7 +37,7 @@ int _wkb_ibus_log_dom = -1; return; \ } \ DBG("Message '%s' with signature '%s'", eldbus_message_member_get(_msg), eldbus_message_signature_get(_msg)); \ - } while (0); + } while (0) struct _wkb_ibus_service { @@ -70,7 +70,7 @@ _wkb_config_value_changed_cb(void *data, const Eldbus_Message *msg) const char *section, name; Eldbus_Message_Iter *value; - CHECK_MESSAGE_ERRORS(msg) + _check_message_errors(msg); if (!eldbus_message_arguments_get(msg, "ssv", §ion, &name, &value)) { @@ -123,7 +123,7 @@ _wkb_name_acquired_cb(void *data, const Eldbus_Message *msg) DBG("NameAcquired"); - CHECK_MESSAGE_ERRORS(msg) + _check_message_errors(msg); if (!eldbus_message_arguments_get(msg, "s", &name)) { @@ -170,7 +170,7 @@ _wkb_name_lost_cb(void *data, const Eldbus_Message *msg) DBG("NameLost"); - CHECK_MESSAGE_ERRORS(msg) + _check_message_errors(msg); if (!eldbus_message_arguments_get(msg, "s", &name)) { @@ -226,7 +226,7 @@ _wkb_name_release_cb(void *data, const Eldbus_Message *msg, Eldbus_Pending *pend { unsigned int reply; - CHECK_MESSAGE_ERRORS(msg) + _check_message_errors(msg); if (!eldbus_message_arguments_get(msg, "u", &reply)) { |