config.h must be included or visibility stuff fails
[libfirm] / ir / obstack / obstack_printf.c
index a8b5a98..a270fea 100644 (file)
@@ -1,3 +1,4 @@
+#include <config.h>
 #include <stdarg.h>
 #include <stdio.h>
 #include <stdlib.h>
@@ -5,10 +6,16 @@
 #include "obstack.h"
 
 #ifdef _WIN32
-#define vsnprintf _vsnprintf
-#endif
-
-int obstack_vprintf(struct obstack *obst, const char *fmt, va_list ap)
+/* win32/C89 has no va_copy function... so we have to use the stupid fixed-length version */
+int obstack_vprintf(struct obstack *obst, const char *fmt, va_list ap) FIRM_NOTHROW
+{
+       char buf[16384];
+       int len = _vsnprintf(buf, sizeof(buf), fmt, ap);
+       obstack_grow(obst, buf, len);
+       return len;
+}
+#else
+int obstack_vprintf(struct obstack *obst, const char *fmt, va_list ap) FIRM_NOTHROW
 {
        char    buf[128];
        char   *buffer = buf;
@@ -16,7 +23,9 @@ int obstack_vprintf(struct obstack *obst, const char *fmt, va_list ap)
        int     len;
 
        for (;;) {
-               len = vsnprintf(buffer, sizeof(buffer), fmt, ap);
+               va_list tap;
+               va_copy(tap, ap);
+               len = vsnprintf(buffer, size, fmt, tap);
 
                /* snprintf should return -1 only in the error case, but older glibcs
                 * and probably other systems are buggy in this respect and return -1 if
@@ -35,7 +44,7 @@ int obstack_vprintf(struct obstack *obst, const char *fmt, va_list ap)
                } else {
                        break;
                }
-               buffer = malloc(size);
+               buffer = (char*)malloc(size);
        }
 
        obstack_grow(obst, buffer, len);
@@ -44,8 +53,9 @@ int obstack_vprintf(struct obstack *obst, const char *fmt, va_list ap)
 
        return len;
 }
+#endif
 
-int obstack_printf(struct obstack *obst, const char *fmt, ...)
+int obstack_printf(struct obstack *obst, const char *fmt, ...) FIRM_NOTHROW
 {
        va_list ap;
        int     res;