After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 317458 - atkoobject.c: ATK_LOCALEDIR should be locale character encoding
atkoobject.c: ATK_LOCALEDIR should be locale character encoding
Status: RESOLVED FIXED
Product: atk
Classification: Platform
Component: general
1.10.x
Other All
: Normal minor
: ---
Assigned To: bill.haneman
bill.haneman
Depends on:
Blocks:
 
 
Reported: 2005-09-28 18:22 UTC by Kazuki Iwamoto
Modified: 2006-02-24 14:10 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
fix get_atk_locale_dir (690 bytes, patch)
2005-09-28 18:23 UTC, Kazuki Iwamoto
committed Details | Review

Description Kazuki Iwamoto 2005-09-28 18:22:23 UTC
Please describe the problem:
ATK_LOCALEDIR is UTF-8.
But, the second argument of bindtextdomain is locale character encoding.

Steps to reproduce:
1. put files on the folder which has non-ascii character.
2. run any program.


Actual results:
Strings are not translated.

Expected results:
Convert the reesult from g_win32_get_package_installation_subdirectory by using
g_win32_locale_filename_from_utf8 at get_atk_locale_dir.

Does this happen every time?
every time

Other information:
Comment 1 Kazuki Iwamoto 2005-09-28 18:23:25 UTC
Created attachment 52784 [details] [review]
fix get_atk_locale_dir
Comment 2 Kazuki Iwamoto 2005-09-28 18:40:35 UTC
This bug occurs on Win32 only.

Since there is same bug in GTK+, I have reported simultaneously.
http://bugzilla.gnome.org/show_bug.cgi?id=317457
Comment 3 Kazuki Iwamoto 2005-09-28 19:21:31 UTC
Comment on attachment 52784 [details] [review]
fix get_atk_locale_dir

>--- atkobject.c.orig	2005-08-25 03:01:25.000000000 +0900
>+++ atkobject.c	2005-09-29 03:13:53.000000000 +0900
>@@ -232,11 +232,21 @@
> 
> G_WIN32_DLLMAIN_FOR_DLL_NAME(static, dll_name)
> 
>-static char *
>+static const char *
> get_atk_locale_dir (void)
> {
>-  return g_win32_get_package_installation_subdirectory
>-    (GETTEXT_PACKAGE, dll_name, "lib/locale");
>+  static char *atk_localedir = NULL;
>+
>+  if (!atk_localedir)
>+    {
>+      gchar *temp;
>+
>+      temp = g_win32_get_package_installation_subdirectory
>+        (GETTEXT_PACKAGE, dll_name, "lib\\locale");
>+      atk_localedir = g_win32_locale_filename_from_utf8 (temp);
>+      g_free (temp);
>+    }
>+  return atk_localedir;
> }
> 
> #endif
Comment 4 bill.haneman 2006-02-24 14:10:31 UTC
Thanks!