branch master updated: * tp/Texinfo/XS/main/translations.c (translate_string): Remove commented-out code, the purpose of which is not evident. Remove FIXME comment as this should be working now we call switch_to_global_locale.

[Gavin D. Smith]

This is an automated email from the git hooks/post-receive script.

gavin pushed a commit to branch master
in repository texinfo.

The following commit(s) were added to refs/heads/master by this push:
     new ad008816f1 * tp/Texinfo/XS/main/translations.c (translate_string): 
Remove commented-out code, the purpose of which is not evident. Remove FIXME 
comment as this should be working now we call switch_to_global_locale.
ad008816f1 is described below

commit ad008816f1b27cac7de60304ee7a727c00ee2a3a
Author: Gavin Smith <gavinsmith0123@gmail.com>
AuthorDate: Sat Nov 18 10:21:31 2023 +0000

    * tp/Texinfo/XS/main/translations.c (translate_string):
    Remove commented-out code, the purpose of which is not evident.
    Remove FIXME comment as this should be working now we call
    switch_to_global_locale.
---
 ChangeLog                         |  7 +++++++
 tp/Texinfo/XS/main/translations.c | 21 +--------------------
 2 files changed, 8 insertions(+), 20 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index 2accfcda55..07a946752b 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,10 @@
+2023-11-18  Gavin Smith <gavinsmith0123@gmail.com>
+
+       * tp/Texinfo/XS/main/translations.c (translate_string):
+       Remove commented-out code, the purpose of which is not evident.
+       Remove FIXME comment as this should be working now we call
+       switch_to_global_locale.
+
 2023-11-18  Gavin Smith <gavinsmith0123@gmail.com>
 
        * tp/Texinfo/XS/main/translations.c (translate_string):
diff --git a/tp/Texinfo/XS/main/translations.c 
b/tp/Texinfo/XS/main/translations.c
index 99477990ef..de4ed82e03 100644
--- a/tp/Texinfo/XS/main/translations.c
+++ b/tp/Texinfo/XS/main/translations.c
@@ -174,11 +174,6 @@ translate_string (OPTIONS *options, const char * string,
       lang = "en";
     }
 
-
-  /*
-  return strdup (string);
-   */
-
 #ifndef ENABLE_NLS
   translated_string = strdup (string);
   return translated_string;
@@ -209,10 +204,7 @@ translate_string (OPTIONS *options, const char * string,
     list through the LANGUAGE variable.
 
    We set LANG and then LC_MESSAGES to a valid locale in
-   switch_messages_locale to have LANGUAGE work in that case.
-   FIXME it does not work.  Also tested with setting LC_ALL instead of
-         LC_MESSAGES and it does not work either
-   */
+   switch_messages_locale to have LANGUAGE work in that case. */
 
   saved_LANG = getenv ("LANG");
 
@@ -240,17 +232,6 @@ translate_string (OPTIONS *options, const char * string,
   textdomain (strings_textdomain);
   bind_textdomain_codeset (strings_textdomain, "utf-8");
 
-  /*
-   {
-     char *bindtextdomain_dir;
-     char *current_textdomain;
-     current_textdomain = textdomain (NULL);
-     bindtextdomain_dir = bindtextdomain (current_textdomain, NULL);
-     fprintf (stderr, "DOMAIN %s '%s'\n", current_textdomain,
-                                         bindtextdomain_dir);
-   }
-   */
-
   langs[0] = strdup (lang);
   p = strchr (lang, '_');
   if (p && p - lang > 0)