refactor: simplify utility functions

Author: kanru

po/POTFILES

@@ -8,4 +8,3 @@ src/ibus-chewing-lookup-table.c
 src/ibus-chewing-preedit.c
 src/ibus-chewing-util.c
 src/main.c
-src/maker-dialog-util.c

src/CMakeLists.txt

@@ -1,9 +1,7 @@
 add_library(common STATIC
     ibus-chewing-lookup-table.c
-    maker-dialog-util.c
 
     ibus-chewing-lookup-table.h
-    maker-dialog-util.h
     ibus-chewing-engine.h
 )
 target_link_libraries(common PUBLIC

src/ibus-chewing-engine.c

@@ -23,7 +23,6 @@
 #include "ibus-chewing-preedit.h"
 #include "ibus-chewing-lookup-table.h"
 #include "ibus-chewing-util.h"
-#include "maker-dialog-util.h"
 #include "ibus-chewing-engine-private.h"
 #include <chewing.h>
 #include <glib.h>
@@ -569,7 +568,7 @@ void ibus_chewing_engine_refresh_property(IBusChewingEngine *self,
 #ifndef UNIT_TEST
         IBUS_CHEWING_LOG(DEBUG, "refresh_property(%s) status=%x", prop_name, self->statusFlags);
 
-        if (STRING_EQUALS(prop_name, "InputMode")) {
+        if (g_strcmp0(prop_name, "InputMode") == 0) {
 
             ibus_property_set_label(self->InputMode,
                                     ibus_chewing_pre_edit_get_chi_eng_mode(self->icPreEdit)
@@ -585,7 +584,7 @@ void ibus_chewing_engine_refresh_property(IBusChewingEngine *self,
 
             ibus_engine_update_property(IBUS_ENGINE(self), self->InputMode);
 
-        } else if (STRING_EQUALS(prop_name, "AlnumSize")) {
+        } else if (g_strcmp0(prop_name, "AlnumSize") == 0) {
 
             ibus_property_set_label(self->AlnumSize, chewing_get_ShapeMode(self->icPreEdit->context)
                                                          ? self->AlnumSize_label_full
@@ -601,7 +600,7 @@ void ibus_chewing_engine_refresh_property(IBusChewingEngine *self,
             if (self->statusFlags & ENGINE_FLAG_PROPERTIES_REGISTERED)
                 ibus_engine_update_property(IBUS_ENGINE(self), self->AlnumSize);
 
-        } else if (STRING_EQUALS(prop_name, "setup_prop")) {
+        } else if (g_strcmp0(prop_name, "setup_prop") == 0) {
 #if IBUS_CHECK_VERSION(1, 5, 0)
             ibus_property_set_symbol(self->setup_prop, self->setup_prop_symbol);
 #endif
@@ -650,11 +649,11 @@ static IBusProperty *ibus_chewing_engine_get_ibus_property_by_name(IBusChewingEn
     g_return_val_if_fail(self != NULL, (IBusProperty *)0);
     g_return_val_if_fail(IBUS_IS_CHEWING_ENGINE(self), (IBusProperty *)0);
     {
-        if (STRING_EQUALS(prop_name, "InputMode")) {
+        if (g_strcmp0(prop_name, "InputMode") == 0) {
             return self->InputMode;
-        } else if (STRING_EQUALS(prop_name, "AlnumSize")) {
+        } else if (g_strcmp0(prop_name, "AlnumSize") == 0) {
             return self->AlnumSize;
-        } else if (STRING_EQUALS(prop_name, "setup_prop")) {
+        } else if (g_strcmp0(prop_name, "setup_prop") == 0) {
             return self->setup_prop;
         }
         IBUS_CHEWING_LOG(MSG, "get_ibus_property_by_name(%s): NULL is returned", prop_name);
@@ -1017,7 +1016,7 @@ void commit_text(IBusChewingEngine *self) {
 gboolean ibus_chewing_engine_process_key_event(IBusEngine *engine, KSym keySym, guint keycode,
                                                KeyModifiers unmaskedMod) {
     IBUS_CHEWING_LOG(MSG, "******** process_key_event(-,%x(%s),%x,%x) %s", keySym,
-                     key_sym_get_name(keySym), keycode, unmaskedMod,
+                     ibus_keyval_name(keySym), keycode, unmaskedMod,
                      modifiers_to_string(unmaskedMod));
 
     IBusChewingEngine *self = IBUS_CHEWING_ENGINE(engine);
@@ -1081,17 +1080,17 @@ void ibus_chewing_engine_property_activate(IBusEngine *engine, const gchar *prop
     IBUS_CHEWING_LOG(INFO, "property_activate(-, %s, %u)", prop_name, prop_state);
     IBusChewingEngine *self = IBUS_CHEWING_ENGINE(engine);
 
-    if (STRING_EQUALS(prop_name, "InputMode")) {
+    if (g_strcmp0(prop_name, "InputMode") == 0) {
         /* Toggle Chinese <-> English */
         ibus_chewing_pre_edit_toggle_chi_eng_mode(self->icPreEdit);
         IBUS_CHEWING_LOG(INFO, "property_activate chinese=%d", is_chinese_mode(self));
         ibus_chewing_engine_refresh_property(self, prop_name);
-    } else if (STRING_EQUALS(prop_name, "AlnumSize")) {
+    } else if (g_strcmp0(prop_name, "AlnumSize") == 0) {
         /* Toggle Full <-> Half */
         ibus_chewing_pre_edit_toggle_full_half_mode(self->icPreEdit);
         IBUS_CHEWING_LOG(INFO, "property_activate fullwidth=%d", is_fullwidth_mode(self));
         ibus_chewing_engine_refresh_property(self, prop_name);
-    } else if (STRING_EQUALS(prop_name, "setup_prop")) {
+    } else if (g_strcmp0(prop_name, "setup_prop") == 0) {
         /* open preferences window */
         char *argv[] = {QUOTE_ME(LIBEXEC_DIR) "/ibus-setup-chewing", NULL};
         g_spawn_async(NULL, argv, NULL, G_SPAWN_DEFAULT, NULL, NULL, NULL, NULL);
@@ -1103,15 +1102,15 @@ void ibus_chewing_engine_property_activate(IBusEngine *engine, const gchar *prop
 
 char ibus_chewing_engine_get_default_english_case(IBusChewingEngine *self) {
     char *prop = self->prop_default_english_case;
-    return STRING_EQUALS(prop, "lowercase") ? 'l' : STRING_EQUALS(prop, "uppercase") ? 'u' : 'n';
+    return g_strcmp0(prop, "lowercase") == 0 ? 'l' : g_strcmp0(prop, "uppercase") == 0 ? 'u' : 'n';
 }
 
 char ibus_chewing_engine_get_chinese_english_toggle_key(IBusChewingEngine *self) {
     char *prop = self->prop_chi_eng_mode_toggle;
-    return STRING_EQUALS(prop, "caps_lock") ? 'c'
-           : STRING_EQUALS(prop, "shift")   ? 's'
-           : STRING_EQUALS(prop, "shift_l") ? 'l'
-           : STRING_EQUALS(prop, "shift_r") ? 'r'
+    return g_strcmp0(prop, "caps_lock") == 0 ? 'c'
+           : g_strcmp0(prop, "shift")   == 0 ? 's'
+           : g_strcmp0(prop, "shift_l") == 0 ? 'l'
+           : g_strcmp0(prop, "shift_r") == 0 ? 'r'
                                             : 'n';
     return 'n';
 }

src/ibus-chewing-lookup-table.c

@@ -1,6 +1,6 @@
 #include "ibus-chewing-lookup-table.h"
 #include "ibus-chewing-util.h"
-#include "maker-dialog-util.h"
+#include <ctype.h>
 
 IBusLookupTable *ibus_chewing_lookup_table_new(ChewingContext *context) {
     guint size = 10;

src/ibus-chewing-preedit-private.h

@@ -76,7 +76,7 @@
 
 #define handle_log(funcName)                                                                       \
     IBUS_CHEWING_LOG(INFO, "* self_handle_%s(-,%x(%s),%x(%s))", funcName, kSym,                    \
-                     key_sym_get_name(kSym), unmaskedMod, modifiers_to_string(unmaskedMod));
+                     ibus_keyval_name(kSym), unmaskedMod, modifiers_to_string(unmaskedMod));
 
 KSym self_key_sym_fix(IBusChewingPreEdit *self, KSym kSym, KeyModifiers unmaskedMod);
 

src/ibus-chewing-preedit.c

@@ -2,8 +2,9 @@
 #include "ibus-chewing-preedit.h"
 #include "ibus-chewing-lookup-table.h"
 #include "ibus-chewing-util.h"
-#include "maker-dialog-util.h"
 #include "ibus-chewing-preedit-private.h"
+#include "ibus.h"
+#include <ctype.h>
 #include <chewing.h>
 #include <glib.h>
 
@@ -258,7 +259,7 @@ EventResponse self_handle_key_sym_default(IBusChewingPreEdit *self, KSym kSym,
     KSym fixedKSym = self_key_sym_fix(self, kSym, unmaskedMod);
 
     IBUS_CHEWING_LOG(DEBUG, "* self_handle_key_sym_default(): new kSym %x(%s), %x(%s)", fixedKSym,
-                     key_sym_get_name(fixedKSym), unmaskedMod, modifiers_to_string(unmaskedMod));
+                     ibus_keyval_name(fixedKSym), unmaskedMod, modifiers_to_string(unmaskedMod));
     gint ret = chewing_handle_Default(self->context, fixedKSym);
 
     /* Handle quick commit */
@@ -770,7 +771,7 @@ gboolean is_shift_toggle(KSym keyLast, KSym kSym, KeyModifiers unmaskedMod) {
 gboolean ibus_chewing_pre_edit_process_key(IBusChewingPreEdit *self, KSym kSym,
                                            KeyModifiers unmaskedMod) {
     IBUS_CHEWING_LOG(INFO, "***** ibus_chewing_pre_edit_process_key(-,%x(%s),%x(%s))", kSym,
-                     key_sym_get_name(kSym), unmaskedMod, modifiers_to_string(unmaskedMod));
+                     ibus_keyval_name(kSym), unmaskedMod, modifiers_to_string(unmaskedMod));
     process_key_debug("Before response");
 
     /* Find corresponding rule */