Remove PerlIO_sprintf() and PerlIO_vsprintf()
authorKarl Williamson <public@khwilliamson.com>
Mon, 7 Oct 2013 04:14:45 +0000 (22:14 -0600)
committerKarl Williamson <public@khwilliamson.com>
Mon, 7 Oct 2013 04:18:01 +0000 (22:18 -0600)
http://markmail.org/message/m6bvkxvqnvjnfgyf

perlio.c
pod/perldelta.pod

index 2591f89..279b529 100644 (file)
--- a/perlio.c
+++ b/perlio.c
@@ -5183,36 +5183,6 @@ vfprintf(FILE *fd, char *pat, char *args)
 
 #endif
 
-#ifndef PerlIO_vsprintf
-int
-PerlIO_vsprintf(char *s, int n, const char *fmt, va_list ap)
-{
-    dTHX; 
-    const int val = my_vsnprintf(s, n > 0 ? n : 0, fmt, ap);
-    PERL_UNUSED_CONTEXT;
-
-#ifndef PERL_MY_VSNPRINTF_GUARDED
-    if (val < 0 || (n > 0 ? val >= n : 0)) {
-       Perl_croak(aTHX_ "panic: my_vsnprintf overflow in PerlIO_vsprintf\n");
-    }
-#endif
-    return val;
-}
-#endif
-
-#ifndef PerlIO_sprintf
-int
-PerlIO_sprintf(char *s, int n, const char *fmt, ...)
-{
-    va_list ap;
-    int result;
-    va_start(ap, fmt);
-    result = PerlIO_vsprintf(s, n, fmt, ap);
-    va_end(ap);
-    return result;
-}
-#endif
-
 /*
  * Local variables:
  * c-indentation-style: bsd
index 1da12d8..9c8531b 100644 (file)
@@ -78,6 +78,11 @@ XXX For a release on a stable branch, this section aspires to be:
 
 [ List each incompatible change as a =head2 entry ]
 
+=head2 Functions C<PerlIO_vsprintf> and C<PerlIO_sprintf> have been removed
+
+These two functions, undocumented, unused in CPAN, and problematic have
+been removed.
+
 =head1 Deprecations
 
 XXX Any deprecated features, syntax, modules etc. should be listed here.