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);
|
