diff options
author | Bruno Haible <bruno@clisp.org> | 2023-03-24 11:17:22 +0100 |
---|---|---|
committer | Bruno Haible <bruno@clisp.org> | 2023-03-24 11:17:22 +0100 |
commit | 21cb2b8ed03e0dd628c9be30734f563627a9b7e5 (patch) | |
tree | 69fbcf313a382556f47992719c1a3467562a846f | |
parent | 0bba8aca9171f8b2e8835b0e2733541e8f119496 (diff) | |
download | gnulib-21cb2b8ed03e0dd628c9be30734f563627a9b7e5.tar.gz |
vsprintf-gnu: New module.
* m4/vsprintf-posix.m4 (gl_FUNC_VSPRINTF_IS_POSIX): New macro,
extracted from gl_FUNC_VSPRINTF_POSIX.
(gl_FUNC_VSPRINTF_POSIX): Require it. Invoke
gl_PREREQ_VASNPRINTF_WITH_POSIX_EXTRAS.
* m4/vsprintf-gnu.m4: New file, based on m4/vsprintf-posix.m4.
* modules/vsprintf-gnu: New file, based on modules/vasnprintf-gnu.
* doc/posix-functions/vsprintf.texi: Mention the new module.
-rw-r--r-- | ChangeLog | 11 | ||||
-rw-r--r-- | doc/posix-functions/vsprintf.texi | 11 | ||||
-rw-r--r-- | m4/vsprintf-gnu.m4 | 25 | ||||
-rw-r--r-- | m4/vsprintf-posix.m4 | 31 | ||||
-rw-r--r-- | modules/vsprintf-gnu | 28 |
5 files changed, 86 insertions, 20 deletions
@@ -1,5 +1,16 @@ 2023-03-24 Bruno Haible <bruno@clisp.org> + vsprintf-gnu: New module. + * m4/vsprintf-posix.m4 (gl_FUNC_VSPRINTF_IS_POSIX): New macro, + extracted from gl_FUNC_VSPRINTF_POSIX. + (gl_FUNC_VSPRINTF_POSIX): Require it. Invoke + gl_PREREQ_VASNPRINTF_WITH_POSIX_EXTRAS. + * m4/vsprintf-gnu.m4: New file, based on m4/vsprintf-posix.m4. + * modules/vsprintf-gnu: New file, based on modules/vasnprintf-gnu. + * doc/posix-functions/vsprintf.texi: Mention the new module. + +2023-03-24 Bruno Haible <bruno@clisp.org> + snprintf-gnu: Add tests. * tests/test-snprintf-gnu.c: New file, based on tests/test-snprintf-posix.c. diff --git a/doc/posix-functions/vsprintf.texi b/doc/posix-functions/vsprintf.texi index 27ec5b9141..5ae3644e00 100644 --- a/doc/posix-functions/vsprintf.texi +++ b/doc/posix-functions/vsprintf.texi @@ -4,9 +4,9 @@ POSIX specification:@* @url{https://pubs.opengroup.org/onlinepubs/9699919799/functions/vsprintf.html} -Gnulib module: vsprintf-posix +Gnulib module: vsprintf-posix or vsprintf-gnu -Portability problems fixed by Gnulib: +Portability problems fixed by either Gnulib module @code{vsprintf-posix} or @code{vsprintf-gnu}: @itemize @item This function does not support size specifiers as in C99 (@code{hh}, @code{ll}, @@ -80,6 +80,13 @@ The compiler warns that this function is deprecated: macOS 13.0. @end itemize +Portability problems fixed by Gnulib module @code{vsprintf-gnu}: +@itemize +@item +This function does not support the @samp{B} directive on some platforms: +glibc 2.34, FreeBSD 13.1, NetBSD 9.0, OpenBSD 7.2, macOS 12.5, AIX 7.2, Solaris 11.4, and others. +@end itemize + Portability problems not fixed by Gnulib: @itemize @item diff --git a/m4/vsprintf-gnu.m4 b/m4/vsprintf-gnu.m4 new file mode 100644 index 0000000000..4569e3599b --- /dev/null +++ b/m4/vsprintf-gnu.m4 @@ -0,0 +1,25 @@ +# vsprintf-gnu.m4 serial 1 +dnl Copyright (C) 2023 Free Software Foundation, Inc. +dnl This file is free software; the Free Software Foundation +dnl gives unlimited permission to copy and/or distribute it, +dnl with or without modifications, as long as this notice is preserved. + +AC_DEFUN([gl_FUNC_VSPRINTF_GNU], +[ + AC_REQUIRE([gl_FUNC_VSPRINTF_IS_POSIX]) + AC_REQUIRE([gl_PRINTF_DIRECTIVE_UPPERCASE_B]) + gl_cv_func_vsprintf_gnu=no + if test $gl_cv_func_vsprintf_posix = yes; then + case "$gl_cv_func_printf_directive_uppercase_b" in + *yes) + # vsprintf is already POSIX compliant and GNU compatible. + gl_cv_func_vsprintf_gnu=yes + ;; + esac + fi + if test $gl_cv_func_vsprintf_gnu = no; then + gl_PREREQ_VASNPRINTF_WITH_GNU_EXTRAS + gl_REPLACE_VASNPRINTF + gl_REPLACE_VSPRINTF + fi +]) diff --git a/m4/vsprintf-posix.m4 b/m4/vsprintf-posix.m4 index 94ef61e9dd..bc9e19f99a 100644 --- a/m4/vsprintf-posix.m4 +++ b/m4/vsprintf-posix.m4 @@ -1,4 +1,4 @@ -# vsprintf-posix.m4 serial 14 +# vsprintf-posix.m4 serial 15 dnl Copyright (C) 2007-2023 Free Software Foundation, Inc. dnl This file is free software; the Free Software Foundation dnl gives unlimited permission to copy and/or distribute it, @@ -6,6 +6,18 @@ dnl with or without modifications, as long as this notice is preserved. AC_DEFUN([gl_FUNC_VSPRINTF_POSIX], [ + AC_REQUIRE([gl_FUNC_VSPRINTF_IS_POSIX]) + if test $gl_cv_func_vsprintf_posix = no; then + gl_PREREQ_VASNPRINTF_WITH_POSIX_EXTRAS + gl_REPLACE_VASNPRINTF + gl_REPLACE_VSPRINTF + fi +]) + +dnl Test whether vsprintf is POSIX compliant. +dnl Result is gl_cv_func_vsprintf_posix. +AC_DEFUN([gl_FUNC_VSPRINTF_IS_POSIX], +[ AC_REQUIRE([gl_PRINTF_SIZES_C99]) AC_REQUIRE([gl_PRINTF_LONG_DOUBLE]) AC_REQUIRE([gl_PRINTF_INFINITE]) @@ -90,23 +102,6 @@ AC_DEFUN([gl_FUNC_VSPRINTF_POSIX], esac ;; esac - if test $gl_cv_func_vsprintf_posix = no; then - gl_PREREQ_VASNPRINTF_LONG_DOUBLE - gl_PREREQ_VASNPRINTF_INFINITE_DOUBLE - gl_PREREQ_VASNPRINTF_INFINITE_LONG_DOUBLE - gl_PREREQ_VASNPRINTF_DIRECTIVE_A - gl_PREREQ_VASNPRINTF_DIRECTIVE_B - gl_PREREQ_VASNPRINTF_DIRECTIVE_F - gl_PREREQ_VASNPRINTF_DIRECTIVE_LS - gl_PREREQ_VASNPRINTF_DIRECTIVE_LC - gl_PREREQ_VASNPRINTF_FLAG_GROUPING - gl_PREREQ_VASNPRINTF_FLAG_LEFTADJUST - gl_PREREQ_VASNPRINTF_FLAG_ZERO - gl_PREREQ_VASNPRINTF_PRECISION - gl_PREREQ_VASNPRINTF_ENOMEM - gl_REPLACE_VASNPRINTF - gl_REPLACE_VSPRINTF - fi ]) AC_DEFUN([gl_REPLACE_VSPRINTF], diff --git a/modules/vsprintf-gnu b/modules/vsprintf-gnu new file mode 100644 index 0000000000..d86b062e2e --- /dev/null +++ b/modules/vsprintf-gnu @@ -0,0 +1,28 @@ +Description: +POSIX and GNU compatible vsprintf() function: print formatted output to a string + +Comment: +This module should not be used as a dependency from a test module, +otherwise when this module occurs as a tests-related module, it will +have side effects on the compilation of the 'vasnprintf' module, if +that module occurs among the main modules in lib/. + +Files: +m4/vsprintf-gnu.m4 + +Depends-on: +vsprintf-posix + +configure.ac: +gl_FUNC_VSPRINTF_GNU + +Makefile.am: + +Include: +<stdio.h> + +License: +LGPL + +Maintainer: +all |