removed this function which was not publically exported in glib.h. to

Mon Aug 24 02:08:56 1998  Tim Janik  <timj@gtk.org>

        * glib.h:
        * gstring.c:
        * gstrfuncs.c:
        (g_vsprintf): removed this function which was not publically
        exported in glib.h. to export it, it should have been named
        differently in the first place, since its semantics differ from
        vsprintf(). apart from that, it was a possible cause for
        problems since it worked on a previously allocated memory area and
        was used in a lot places of glib. exporting it would have been a
        guararant for problems with threaded programs.
        (g_printf_string_upper_bound): exported this function to return
        a string size, guarranteed to be big enough to hold the fully
        expanded format+args string. added 'q', 'L' and 'll' flag handling.
        in fact, the newly allocated area is in most cases much bigger than
        required.
        (g_strdup_vprintf()): new function returning a newly allocated string
        containing the contents of *format and associated args (size is
        calculated with g_printf_string_upper_bound()).
        (g_strdup_printf): new function which wraps g_strdup_vprintf().

        * configure.in: check for va_copy() or __va_copy() alternatively.
        check whether va_lists can be copyied by value.

        * glib.h: provide a definition for G_VA_COPY.

        * glib.h:
        * gmessages.c:
        (g_logv):
        (g_vsnprintf):
        pass va_lists by value, not by reference, since this causes problems
        on platforms that implement va_list as as arrays. internaly, use
        G_VA_COPY (new_arg, org_arg); va_end (new_arg); to produce a second
        va_list variable, if multiple passes are required. changed all
        callers.

        * glib.h:
        * gerror.h:
        renamed g_debug() to g_on_error_query(), cleaned up a bit.
        renamed g_stack_trace() to g_on_error_stack_trace() since both
        functions cluttered different namespaces.
        there is an appropriate comment in glib.h now that explains the
        unix and gdb specific dependencies of both functions.
        removed g_attach_process().
        g_on_error_stack_trace() should probably be handled with caution,
        i've seem several different linux versions (2.0.x) become unstable
        after invokation of this function.

1 files changed, 4 insertions, 13 deletions

diff --git a/gscanner.c b/gscanner.c
index aa0239554..f25b11d0c 100644
--- a/gscanner.c
+++ b/gscanner.c
@@ -102,7 +102,6 @@ static	GScannerConfig	g_scanner_config_template =
 
 
 /* --- prototypes --- */
-extern char* g_vsprintf (gchar *fmt, va_list *args, va_list *args2);
 static inline
 gint		g_scanner_char_2_num	  (guchar	 c,
 					   guchar	 base);
@@ -271,16 +270,12 @@ g_scanner_error (GScanner	*scanner,
   
   if (scanner->msg_handler)
     {
-      va_list args, args2;
+      va_list args;
       gchar *string;
       
       va_start (args, format);
-      va_start (args2, format);
-      string = g_vsprintf ((gchar*) format, &args, &args2);
+      string = g_strdup_vprintf (format, args);
       va_end (args);
-      va_end (args2);
-      
-      string = g_strdup (string);
       
       scanner->msg_handler (scanner, string, TRUE);
       
@@ -298,16 +293,12 @@ g_scanner_warn (GScanner       *scanner,
   
   if (scanner->msg_handler)
     {
-      va_list args, args2;
+      va_list args;
       gchar *string;
       
       va_start (args, format);
-      va_start (args2, format);
-      string = g_vsprintf ((gchar*) format, &args, &args2);
+      string = g_strdup_vprintf (format, args);
       va_end (args);
-      va_end (args2);
-      
-      string = g_strdup (string);
       
       scanner->msg_handler (scanner, string, FALSE);