argv_localedir

[Bruno H. <br...@cl...>]

Hi Sam,

What does this patch mean?

1) bindtextdomain("...", NULL) is defined to return the old localedir for that
   domain. It must not crash. If it does, report it to the guy responsible for
   this implementation of bindtextdomain().

2) If bindtextdomain("...", NULL) were to crash, why do you work around it
   only on WIN32_NATIVE? What about the other platforms (not UNIX, not
   WIN32_NATIVE)?

Bruno


diff -r -c3 -w --exclude=CVS --exclude='*.po*' clisp-20050514/src/spvw_language.d clisp-20050527/src/spvw_language.d
*** clisp-20050514/src/spvw_language.d	2005-01-03 12:22:18.000000000 +0100
--- clisp-20050527/src/spvw_language.d	2005-05-27 22:41:49.000000000 +0200
***************
*** 220,225 ****
--- 220,229 ----
          }
        }
      #endif
+    #if defined(WIN32_NATIVE)
+     /* this avoids a segfault on mingw */
+     if (argv_localedir == NULL) argv_localedir = "";
+    #endif
      bindtextdomain("clisp",argv_localedir);
      bindtextdomain("clisplow",argv_localedir);
      if (must_free_argv_localedir) free((void*)argv_localedir);