+2011-02-13 Bruno Haible <bruno@clisp.org>
+
+ mbrtowc: Add more tests for native Windows platforms.
+ * tests/test-mbrtowc-w32-1.sh: New file.
+ * tests/test-mbrtowc-w32-2.sh: New file.
+ * tests/test-mbrtowc-w32-3.sh: New file.
+ * tests/test-mbrtowc-w32-4.sh: New file.
+ * tests/test-mbrtowc-w32-5.sh: New file.
+ * tests/test-mbrtowc-w32.c: New file.
+ * modules/mbrtowc-tests (Files): Add them.
+ (Makefile.am): Arrange to run these tests.
+ * tests/test-mbrtowc-w32-6.sh: New file, currently unused.
+ * tests/test-mbrtowc-w32-7.sh: New file, currently unused.
+
+2011-02-13 Bruno Haible <bruno@clisp.org>
+
+ mbrtowc: Work around native Windows bug.
+ * m4/mbrtowc.m4 (gl_MBRTOWC_RETVAL): Detect native Windows bug. Use the
+ guess when no suitable locale for testing was found.
+ * doc/posix-functions/mbrtowc.texi: Mention the native Windows bug.
+
+2011-02-13 Bruno Haible <bruno@clisp.org>
+
+ mbsinit: Work around mingw bug.
+ * m4/mbsinit.m4 (gl_FUNC_MBSINIT): Replace mbsinit also on mingw.
+ * lib/mbsinit.c (mbsinit): Provide an alternate definition for native
+ Windows.
+ * doc/posix-functions/mbsinit.texi: Mention the mingw bug.
+
+2011-02-13 Bruno Haible <bruno@clisp.org>
+
+ mbsinit: Don't crash for a NULL argument.
+ * lib/mbsinit.c (mbsinit): When the argument is NULL, return 1.
+ * tests/test-mbsinit.c (mbsinit): Check this behaviour.
+
+2011-02-13 Bruno Haible <bruno@clisp.org>
+
+ Don't interfere with a program's definition of __attribute__.
+ * lib/stdio.in.h (__attribute__): Remove definition.
+ (_GL_ATTRIBUTE_FORMAT, _GL_ATTRIBUTE_FORMAT_PRINTF): New macros.
+ (dprintf, fprintf, obstack_printf, obstack_printf, obstack_vprintf,
+ printf, snprintf, sprintf, asprintf, vasprintf, vdprintf, vfprintf,
+ vsnprintf, vsprintf): Use _GL_ATTRIBUTE_FORMAT_PRINTF.
+ * lib/string.in.h (__attribute__): Remove definition.
+ Reported by Paul Eggert.
+
+2011-02-12 Paul Eggert <eggert@cs.ucla.edu>
+
+ stdlib: don't get in the way of non-GCC __attribute__
+ See thread starting at
+ <http://lists.gnu.org/archive/html/bug-gnulib/2011-02/msg00161.html>.
+ Revert previous stdlib change, installing the following instead:
+ * lib/stdlib.in.h (__attribute__): Remove. We do not want
+ to get in the way of a non-GCC compiler that supports __attribute__.
+ (_GL_ATTRIBUTE_RETURN): New macro.
+ (_Exit): Use it instead of __attribute__.
+
+2011-02-12 Bruno Haible <bruno@clisp.org>
+
+ quotearg test: Avoid test failure on mingw.
+ * tests/test-quotearg.sh: Convert the locale identifier from native
+ Windows syntax to Unix syntax.
+
+2011-02-12 Bruno Haible <bruno@clisp.org>
+
+ setlocale: Prefer gnulib's override over libintl's override.
+ * lib/locale.in.h (GNULIB_defined_setlocale): New macro.
+ * lib/gettext.h (setlocale): Redefine to rpl_setlocale if
+ GNULIB_defined_setlocale is set.
+
+2011-02-12 Paul Eggert <eggert@cs.ucla.edu>
+
+ stdlib: support non-GCC __attribute__
+
+ Fix a serious and tricky problem encountered when attempting to
+ add the getloadavg module to Emacs. Emacs worked fine on RHEL
+ 5.5, but it crashed due to memory corruption on Solaris 10 with
+ Sun C 5.11. Emacs normally ORs 3-bit tags into their low-order
+ bits that are otherwise zero. This tagging is optional inside
+ Emacs but is preferred and is used when __attribute__ ((__aligned
+ (8))) works, as it does with both recent-enough GCC and with Sun C
+ 5.11. However, Sun C 5.11 is not GCC and does not #define
+ __GNUC__ and __GNUC_MINOR__.
+
+ When I added the getloadavg module to Emacs, it brought in
+ stdlib.in.h, which contained this fragment:
+
+ #ifndef __attribute__
+ # if __GNUC__ < 2 || (__GNUC__ == 2 && __GNUC_MINOR__ < 8)
+ # define __attribute__(Spec) /* empty */
+ # endif
+ #endif
+
+ When files that include <stdlib.h> were compiled with Sun C 5.11,
+ the above code disabled __attribute__ ((__aligned (8))), which
+ caused variables to not be properly aligned, which eventually led
+ to the pointer corruption mentioned above. (This was a bit hard
+ to diagnose, unfortunately.)
+
+ Several "#define __attribute__(X) /* empty */" code snippets need
+ to be eradicated from Gnulib to work with non-GCC compilers that
+ support __attribute__. The Autoconf way to do this is to test for
+ each kind of attribute that we want support for, and selectively
+ enable that in source code.
+
+ Fix this problem just for stdlib.h, by adding a test for the
+ __noreturn__ attribute, and change stdlib.in.h to use that test
+ when needed. This technique can be easily generalized to the
+ other *.in.h files and attributes, and a similar technique can be
+ used for *.h and *.c files. This patch is enough to solve the
+ problem for Emacs + getloadavg, and I thought I'd publish it for
+ feedback before undertaking further, similar fixes in other
+ modules.
+
+ This patch does not arrange to #define HAVE_ATTRIBUTE_NORETURN
+ because it's not needed for stdlib.h. It merely substitutes the
+ value directly into stdlib.h. We may well need to #define it, or
+ similar symbols, for other modules, but it's nice to also have an
+ option to not #define it for applications like Emacs that do not
+ need it.
+
+ * lib/stdlib.in.h (__attribute__): Do not #define.
+ (_GL_ATTRIBUTE_NORETURN): New macro, which in stdlib.h needs to
+ be defined only if the _Exit module is also used.
+ * m4/_Exit.m4 (gl_FUNC__EXIT): Require gl_ATTRIBUTE_NORETURN.
+ * m4/stdlib_h.m4 (gl_STDLIB_H_DEFAULTS): Subst
+ HAVE_ATTRIBUTE_NORETURN and default it to 1, its value on GNU
+ platforms.
+ * modules/_Exit (Files): Add m4/attribute.m4.
+ * modules/stdlib (Makefile.am): Substitute HAVE_ATTRIBUTE_NORETURN.
+ * m4/attribute.m4: New file.
+
+2011-02-12 Bruno Haible <bruno@clisp.org>
+
+ wcsrtombs: Work around bug on native Windows.
+ * m4/wcsrtombs.m4 (gl_WCSRTOMBS_NULL): Test against mingw bug.
+ * lib/wcsrtombs.c (rpl_wcsrtombs): When dest is NULL, pass SIZE_MAX
+ instead of len.
+ * doc/posix-functions/wcsrtombs.texi: Document mingw bug.
+
+2011-02-12 Bruno Haible <bruno@clisp.org>
+
+ mbsrtowcs: Work around bug on native Windows.
+ * m4/mbsrtowcs.m4 (gl_MBSRTOWCS_WORKS): Require gt_LOCALE_FR. Test
+ against mingw bug.
+ * doc/posix-functions/mbsrtowcs.texi: Document mingw bug.
+
2011-02-12 Bruno Haible <bruno@clisp.org>
Avoid setlocale bugs in tests.
strptime: avoid compiler warnings
* lib/strptime.c (__strptime_internal) [!_NL_CURRENT]: Avoid
compiler warnings about dead code.
- Reported by Matthias Bolte.
+ Reported by Daniel P. Berrange.
2011-02-11 Thien-Thi Nguyen <ttn@gnuvola.org>