From: Bruno Haible <bruno@clisp.org>
Date: Fri, 24 Mar 2023 10:17:22 +0000 (+0100)
Subject: vsprintf-gnu: New module.
X-Git-Tag: v1.0~1593
X-Git-Url: https://gitweb.git.savannah.gnu.org/gitweb/?a=commitdiff_plain;h=21cb2b8ed03e0dd628c9be30734f563627a9b7e5;p=gnulib.git

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.
---

diff --git a/ChangeLog b/ChangeLog
index 826526d567..67b9dd7483 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,14 @@
+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.
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,10 +1,22 @@
-# 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,
 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])
@@ -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