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 679836 - cc-common-language: A few fixes
cc-common-language: A few fixes
Status: RESOLVED WONTFIX
Product: gnome-control-center
Classification: Core
Component: general
unspecified
Other All
: Normal normal
: ---
Assigned To: Control-Center Maintainers
Control-Center Maintainers
Depends on:
Blocks:
 
 
Reported: 2012-07-13 03:36 UTC by Jasper St. Pierre (not reading bugmail)
Modified: 2013-01-11 17:51 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
cc-common-language: Fix memory leaks (1.17 KB, patch)
2012-07-13 03:36 UTC, Jasper St. Pierre (not reading bugmail)
committed Details | Review
cc-common-language: Use standard translations for language names (2.54 KB, patch)
2012-07-13 03:36 UTC, Jasper St. Pierre (not reading bugmail)
needs-work Details | Review

Description Jasper St. Pierre (not reading bugmail) 2012-07-13 03:36:38 UTC
Just a few things I noticed while stealing the code for g-i-s.
Comment 1 Jasper St. Pierre (not reading bugmail) 2012-07-13 03:36:40 UTC
Created attachment 218673 [details] [review]
cc-common-language: Fix memory leaks
Comment 2 Jasper St. Pierre (not reading bugmail) 2012-07-13 03:36:43 UTC
Created attachment 218674 [details] [review]
cc-common-language: Use standard translations for language names
Comment 3 Bastien Nocera 2012-07-13 09:23:59 UTC
Review of attachment 218673 [details] [review]:

Looks fine.
Comment 4 Bastien Nocera 2012-07-13 09:30:18 UTC
Review of attachment 218674 [details] [review]:

This is broken, even after fixing the code to compile, it doesn't show anything in the region panel.

::: panels/common/cc-common-language.c
@@ +515,3 @@
+              char       *name)
+{
+        if (gdm_language_has_translations (locale_id))

cc-common-language.c: In function 'add_language':
cc-common-language.c:517:44: error: 'locale_id' undeclared (first use in this function)
cc-common-language.c:517:44: note: each undeclared identifier is reported only once for each function it appears in

@@ +530,2 @@
         /* Add some common languages first */
+        add_language (ht, "en_US.utf8");

gdm_language_has_translations will fail for the locale with the encoding at the end.
Comment 5 Bastien Nocera 2013-01-11 08:44:54 UTC
Jasper, will the patch be updated?
Comment 6 Jasper St. Pierre (not reading bugmail) 2013-01-11 17:51:53 UTC
Probably not, as it's not that important. The long way isn't too bad.