[gnome-control-center/wip/region-panel: 25/42] common: Cleanup initial languages code



commit 994212d4d58a30858416a53c0d63cf57a99728c9
Author: Rui Matos <tiagomatos gmail com>
Date:   Mon Feb 4 17:04:17 2013 +0100

    common: Cleanup initial languages code
    
    There's no need to use our own translations and setlocale() directly
    since gnome-languages already does that for us.

 panels/common/cc-common-language.c |  104 ++++++++++++++++--------------------
 1 files changed, 47 insertions(+), 57 deletions(-)
---
diff --git a/panels/common/cc-common-language.c b/panels/common/cc-common-language.c
index ac8a777..3e0d272 100644
--- a/panels/common/cc-common-language.c
+++ b/panels/common/cc-common-language.c
@@ -574,55 +574,48 @@ add_other_users_language (GHashTable *ht)
         g_object_unref (proxy);
 }
 
-#define TRANSLATED_LANG(x) g_strdup(dgettext("iso_639", x))
-
 static void
 insert_language (GHashTable *ht,
-		 const char *short_lang,
-		 const char *long_lang,
-		 gboolean    in_iso_639,
-		 const char *lang_label)
+                 const char *lang)
 {
-	const char *lang;
-	char *old_lang;
-	char *label;
-	char *key;
-
-	if (gnome_language_has_translations (long_lang))
-		lang = long_lang;
-	else if (short_lang != NULL && gnome_language_has_translations (short_lang))
-		lang = short_lang;
-	else {
-		g_warning ("%s lacks translations, why is it default?", long_lang);
-		return;
-	}
+        gboolean has_translations;
+        char *label_own_lang;
+        char *label_current_lang;
+        char *label_untranslated;
+        char *key;
+
+        has_translations = gnome_language_has_translations (lang);
+        if (!has_translations) {
+                char *lang_code = g_strndup (lang, 2);
+                has_translations = gnome_language_has_translations (lang_code);
+                g_free (lang_code);
+
+                if (!has_translations)
+                        return;
+        }
 
-	g_debug ("We have translations for %s", lang);
-
-	key = g_strdup_printf ("%s.utf8", long_lang);
-
-	old_lang = g_strdup (setlocale (LC_MESSAGES, NULL));
-	setlocale (LC_MESSAGES, key);
-	g_debug ("Set new lang '%s' (old lang: %s)", key, old_lang);
-	if (in_iso_639)
-		label = dgettext ("iso_639", lang_label);
-	else
-		label = gettext (lang_label);
-	/* We don't have a translation for lang_label in
-	 * its own language? */
-	if (g_strcmp0 (label, lang_label) == 0 &&
-	    g_str_has_prefix (long_lang, "en_") == FALSE) {
-		g_warning ("No translation for %s in %s, trying in current language", lang_label, key);
-		setlocale (LC_MESSAGES, old_lang);
-		g_debug ("Resetting to old lang '%s'", old_lang);
-		label = _(lang_label);
-		g_hash_table_insert (ht, key, g_strdup (label));
-	} else {
-		g_hash_table_insert (ht, key, g_strdup (label));
-		setlocale (LC_MESSAGES, old_lang);
-		g_debug ("Resetting to old lang '%s'", old_lang);
-	}
-	g_free (old_lang);
+        g_debug ("We have translations for %s", lang);
+
+        key = g_strdup_printf ("%s.utf8", lang);
+
+        label_own_lang = gnome_get_language_from_locale (key, key);
+        label_current_lang = gnome_get_language_from_locale (key, NULL);
+        label_untranslated = gnome_get_language_from_locale (key, "C");
+
+        /* We don't have a translation for the label in
+         * its own language? */
+        if (g_strcmp0 (label_own_lang, label_untranslated) == 0) {
+                if (g_strcmp0 (label_current_lang, label_untranslated) == 0)
+                        g_hash_table_insert (ht, key, g_strdup (label_untranslated));
+                else
+                        g_hash_table_insert (ht, key, g_strdup (label_current_lang));
+        } else {
+                g_hash_table_insert (ht, key, g_strdup (label_own_lang));
+        }
+
+        g_free (label_own_lang);
+        g_free (label_current_lang);
+        g_free (label_untranslated);
 }
 
 GHashTable *
@@ -632,18 +625,15 @@ cc_common_language_get_initial_languages (void)
 
         ht = g_hash_table_new_full (g_str_hash, g_str_equal, g_free, g_free);
 
-        /* Add some common languages first,
-         * only the languages not listed in ISO-639 are
-         * marked for translation */
-        insert_language (ht, "en", "en_US", TRUE, "English");
-        insert_language (ht, NULL, "en_GB", FALSE, N_("British English"));
-        insert_language (ht, "de", "de_DE", TRUE, "German");
-        insert_language (ht, "fr", "fr_FR", TRUE, "French");
-        insert_language (ht, "es", "es_ES", FALSE, N_("Spanish"));
-        insert_language (ht, NULL, "zh_CN", FALSE, N_("Chinese (simplified)"));
-        insert_language (ht, "ja", "ja_JP", TRUE, "Japanese");
-        insert_language (ht, "ru", "ru_RU", FALSE, "Russian");
-        insert_language (ht, "ar", "ar_EG", TRUE, "Arabic");
+        insert_language (ht, "en_US");
+        insert_language (ht, "en_GB");
+        insert_language (ht, "de_DE");
+        insert_language (ht, "fr_FR");
+        insert_language (ht, "es_ES");
+        insert_language (ht, "zh_CN");
+        insert_language (ht, "ja_JP");
+        insert_language (ht, "ru_RU");
+        insert_language (ht, "ar_EG");
 
         return ht;
 }


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]