GNOME Bugzilla – Bug 685039
Add nl_langinfo() to libc bindings
Last modified: 2013-11-08 00:58:13 UTC
nl_langinfo() doesn’t currently appear to be bound: http://pubs.opengroup.org/onlinepubs/009604599/functions/nl_langinfo.html
The patch is reasonably easy to make (I am doing this for gcalctool) but I'm not sure what module this should go into. The man page for nl_langinfo says it is in POSIX so I was thinking posix.vapi, but the other locale settings are in glib-2.0.vapi. Let me know where it should go and I'll post a patch.
(In reply to comment #1) > The patch is reasonably easy to make (I am doing this for gcalctool) but I'm > not sure what module this should go into. The man page for nl_langinfo says it > is in POSIX so I was thinking posix.vapi, but the other locale settings are in > glib-2.0.vapi. Let me know where it should go and I'll post a patch. The locale functions in glib-2.0.vapi are all either in GLib or ISO C but nl_langinfo is not, if I'm not mistaken. I would thus suggest to add the binding to posix.vapi. Thanks.
Created attachment 228434 [details] [review] Add nl_langinfo
commit 33f66e1e14058cf0f96b33e3ce882b6117cc22aa Author: Robert Ancell <robert.ancell@canonical.com> Date: Thu Nov 8 14:50:56 2012 +1300 posix: Add nl_langinfo Fixes 685039.
Created attachment 256024 [details] [review] posix: Fix cname of NLItem This fixes the C type of variables of Vala type NLItem. Closes:
A tiny fix is needed to get NLItem working for variables, since its C type currently doesn’t exist (it’s ‘NLItem’ but should be ‘nl_item’).
Ping.
Review of attachment 256024 [details] [review]: Ok as long as it's been tested. Thanks.
Comment on attachment 256024 [details] [review] posix: Fix cname of NLItem commit 7c371598d60cadf7e93695c77f5e949598a66a5c Author: Philip Withnall <philip@tecnocode.co.uk> Date: Sun Sep 29 15:35:07 2013 +0100 posix: Fix cname of NLItem This fixes the C type of variables of Vala type NLItem. Closes: https://bugzilla.gnome.org/show_bug.cgi?id=685039 vapi/posix.vapi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)