]> Savannah Git Hosting - gnulib.git/commitdiff
c-snprintf: Use c-vzsnprintf.
authorBruno Haible <bruno@clisp.org>
Sat, 22 Jun 2024 10:21:35 +0000 (12:21 +0200)
committerBruno Haible <bruno@clisp.org>
Sat, 22 Jun 2024 10:21:35 +0000 (12:21 +0200)
* lib/c-snprintf.h (c_snprintf): Move specification to here.
* lib/c-snprintf.c: Change license to LGPLv2+.
Don't include <stdlib.h>, <string.h>, c-vasnprintf.h. Include
<stdint.h>, c-vsnprintf.h.
(c_snprintf): Implement based on c_vzsnprintf.
* modules/c-snprintf (Depends-on): Add stdint, c-vzsnprintf. Remove
c-vasnprintf.

ChangeLog
lib/c-snprintf.c
lib/c-snprintf.h
modules/c-snprintf

index 83445e3e8c750c74a8a920b3132acc4ddcc2d4ec..fcc60208f597b58f8cd8b3bc09a586d2bbaeb6c3 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,14 @@
 2024-06-22  Bruno Haible  <bruno@clisp.org>
 
+       c-snprintf: Use c-vzsnprintf.
+       * lib/c-snprintf.h (c_snprintf): Move specification to here.
+       * lib/c-snprintf.c: Change license to LGPLv2+.
+       Don't include <stdlib.h>, <string.h>, c-vasnprintf.h. Include
+       <stdint.h>, c-vsnprintf.h.
+       (c_snprintf): Implement based on c_vzsnprintf.
+       * modules/c-snprintf (Depends-on): Add stdint, c-vzsnprintf. Remove
+       c-vasnprintf.
+
        c-zsnprintf: New module.
        * lib/c-snprintf.h: Change license to LGPLv2+.
        (c_zsnprintf): New declaration.
index 8ded5a8fd2879151cf8fd6a9f46283681b3d34f4..658f3faed40a4250820b727bb768a6fde363e46d 100644 (file)
@@ -1,20 +1,18 @@
 /* Formatted output to strings in C locale.
    Copyright (C) 2004, 2006-2024 Free Software Foundation, Inc.
-   Written by Simon Josefsson and Paul Eggert.
-   Modified for C locale by Ben Pfaff.
 
-   This program is free software; you can redistribute it and/or modify
-   it under the terms of the GNU General Public License as published by
-   the Free Software Foundation, either version 3, or (at your option)
-   any later version.
+   This file is free software: you can redistribute it and/or modify
+   it under the terms of the GNU Lesser General Public License as
+   published by the Free Software Foundation; either version 2.1 of the
+   License, or (at your option) any later version.
 
-   This program is distributed in the hope that it will be useful,
+   This file is distributed in the hope that it will be useful,
    but WITHOUT ANY WARRANTY; without even the implied warranty of
    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-   GNU General Public License for more details.
+   GNU Lesser General Public License for more details.
 
-   You should have received a copy of the GNU General Public License along
-   with this program; if not, see <https://www.gnu.org/licenses/>.  */
+   You should have received a copy of the GNU Lesser General Public License
+   along with this program.  If not, see <https://www.gnu.org/licenses/>.  */
 
 #include <config.h>
 
 #include <errno.h>
 #include <limits.h>
 #include <stdarg.h>
-#include <stdlib.h>
-#include <string.h>
+#include <stdint.h>
 
-#include "c-vasnprintf.h"
+#include "c-vsnprintf.h"
 
-/* Print formatted output to string STR.  Similar to sprintf, but
-   additional length SIZE limit how much is written into STR.  Returns
-   string length of formatted string (which may be larger than SIZE).
-   STR may be NULL, in which case nothing will be written.  On error,
-   return a negative value.
-
-   Formatting takes place in the C locale, that is, the decimal point
-   used in floating-point formatting directives is always '.'. */
 int
 c_snprintf (char *str, size_t size, const char *format, ...)
 {
-  char *output;
-  size_t len;
-  size_t lenbuf = size;
   va_list args;
+  ptrdiff_t ret;
 
   va_start (args, format);
-  output = c_vasnprintf (str, &lenbuf, format, args);
-  len = lenbuf;
+  ret = c_vzsnprintf (str, size, format, args);
   va_end (args);
 
-  if (!output)
-    return -1;
-
-  if (output != str)
-    {
-      if (size)
-        {
-          size_t pruned_len = (len < size ? len : size - 1);
-          memcpy (str, output, pruned_len);
-          str[pruned_len] = '\0';
-        }
-
-      free (output);
-    }
-
-  if (INT_MAX < len)
+#if PTRDIFF_MAX > INT_MAX
+  if (ret > INT_MAX)
     {
       errno = EOVERFLOW;
       return -1;
     }
+#endif
 
-  return len;
+  return ret;
 }
index 8e4a6cbe5951930c50c2ad257eaedc57c0afc4d8..f52e058e9ec32a79f06332f3a330c66f59a7ae60 100644 (file)
@@ -48,6 +48,14 @@ extern ptrdiff_t c_zsnprintf (char *restrict str, size_t size,
                               const char *format, ...)
        _GL_ATTRIBUTE_FORMAT ((_GL_ATTRIBUTE_SPEC_PRINTF_STANDARD, 3, 4));
 
+/* Prints formatted output to string STR.  Similar to sprintf, but the
+   additional parameter SIZE limits how much is written into STR.
+   STR may be NULL, in which case nothing will be written.
+   Returns the string length of the formatted string (which may be larger
+   than SIZE).  Upon failure, returns -1 with errno set.
+
+   Formatting takes place in the C locale, that is, the decimal point
+   used in floating-point formatting directives is always '.'. */
 extern int c_snprintf (char *restrict str, size_t size,
                        const char *format, ...)
        _GL_ATTRIBUTE_FORMAT ((_GL_ATTRIBUTE_SPEC_PRINTF_STANDARD, 3, 4));
index 456691d868446673aa513a1865c8497f96fafdc1..49857beca7172bd8e93ffbac046b140db8875425 100644 (file)
@@ -6,8 +6,9 @@ lib/c-snprintf.h
 lib/c-snprintf.c
 
 Depends-on:
+stdint
 stdio
-c-vasnprintf
+c-vzsnprintf
 
 configure.ac:
 AC_REQUIRE([AC_C_RESTRICT])
@@ -19,7 +20,7 @@ Include:
 "c-snprintf.h"
 
 License:
-GPL
+LGPLv2+
 
 Maintainer:
-Ben Pfaff
+all