gEDA-cvs: pcb.git: branch: master updated (aa73c06889aa6c8ac7b7ff5816ee4863915c7994)

[git@xxxxxxxxxx]

The branch, master has been updated
       via  aa73c06889aa6c8ac7b7ff5816ee4863915c7994 (commit)
      from  09aab0bb01e914cc773d2115292695645f561bd5 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


=========
 Summary
=========

 src/hid/gtk/gui-top-window.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)


=================
 Commit Messages
=================

commit aa73c06889aa6c8ac7b7ff5816ee4863915c7994
Author: Peter Clifton <pcjc2@xxxxxxxxx>
Commit: Peter Clifton <pcjc2@xxxxxxxxx>

    hid/gtk: Set the LC_NUMERIC locale to "C" rather than "POSIX"
    
    This value is supported on Win32 platforms, whereas "POSIX" doesn't
    appear to have any effect.
    
    This was one possible way to fix the "zoom doesn't work" bug on Win32
    for locales which use "," as the decimal separator.
    
    DJ fixed the Zoom action to be locale independent in
    commit fd5399c67b988f2f7c9d1a0b4ab7c13bc2e95158, which worked around
    the problem for this particular action, but I feel it would be best
    that we made our locale handling consistent between Linux and Win32
    as well.
    
    Affects-bug: lp-843577

:100644 100644 35bdc69... 98bfcfe... M	src/hid/gtk/gui-top-window.c

=========
 Changes
=========

commit aa73c06889aa6c8ac7b7ff5816ee4863915c7994
Author: Peter Clifton <pcjc2@xxxxxxxxx>
Commit: Peter Clifton <pcjc2@xxxxxxxxx>

    hid/gtk: Set the LC_NUMERIC locale to "C" rather than "POSIX"
    
    This value is supported on Win32 platforms, whereas "POSIX" doesn't
    appear to have any effect.
    
    This was one possible way to fix the "zoom doesn't work" bug on Win32
    for locales which use "," as the decimal separator.
    
    DJ fixed the Zoom action to be locale independent in
    commit fd5399c67b988f2f7c9d1a0b4ab7c13bc2e95158, which worked around
    the problem for this particular action, but I feel it would be best
    that we made our locale handling consistent between Linux and Win32
    as well.
    
    Affects-bug: lp-843577

diff --git a/src/hid/gtk/gui-top-window.c b/src/hid/gtk/gui-top-window.c
index 35bdc69..98bfcfe 100644
--- a/src/hid/gtk/gui-top-window.c
+++ b/src/hid/gtk/gui-top-window.c
@@ -1700,13 +1700,13 @@ ghid_parse_arguments (int *argc, char ***argv)
   /* Do our own setlocale() stufff since we want to override LC_NUMERIC   
    */
   gtk_set_locale ();
-  setlocale (LC_NUMERIC, "POSIX");	/* use decimal point instead of comma */
+  setlocale (LC_NUMERIC, "C");	/* use decimal point instead of comma */
 #endif
 
   /*
    * Prevent gtk_init() and gtk_init_check() from automatically
    * calling setlocale (LC_ALL, "") which would undo LC_NUMERIC if ENABLE_NLS
-   * We also don't want locale set if no ENABLE_NLS to keep POSIX LC_NUMERIC.
+   * We also don't want locale set if no ENABLE_NLS to keep "C" LC_NUMERIC.
    */
   gtk_disable_setlocale ();
 




_______________________________________________
geda-cvs mailing list
geda-cvs@xxxxxxxxxxxxxx
http://www.seul.org/cgi-bin/mailman/listinfo/geda-cvs