Do not use a virtual varargs function

This stuff breaks in some compiler (configurations)

src/BufferedFile.cc

@@ -156,13 +156,9 @@ size_t BufferedFile::transfer(std::ostream& out)
   return count;
 }
 
-int BufferedFile::printf(const char* format, ...)
+int BufferedFile::vprintf(const char* format, va_list va)
 {
-  va_list ap;
+  return vfprintf(fp_, format, va);
-  va_start(ap, format);
-  int r = vfprintf(fp_, format, ap);
-  va_end(ap);
-  return r;
 }
 
 int BufferedFile::flush()

src/BufferedFile.h

@@ -73,8 +73,7 @@ public:
   // Convenient method. Read data to end of file and write them into
   // given stream. Returns written size.
   size_t transfer(std::ostream& out);
-  // wrapper for fprintf
+  virtual int vprintf(const char* format, va_list va);
-  virtual int printf(const char* format, ...);
   // wrapper for fflush
   virtual int flush();
   virtual bool supportsColor();

src/NullOutputFile.h

@@ -43,7 +43,7 @@ class NullOutputFile:public OutputFile {
 public:
   virtual ~NullOutputFile() {}
   virtual size_t write(const char* str) { return 0; }
-  virtual int printf(const char* format, ...) { return 0; }
+  virtual int vprintf(const char* format, va_list va) { return 0; }
   virtual int flush() { return 0; }
   virtual bool supportsColor() { return false; }
 };

src/OutputFile.h

@@ -38,6 +38,7 @@
 #include "common.h"
 
 #include <cstdlib>
+#include <cstdarg>
 
 namespace aria2 {
 
@@ -45,8 +46,15 @@ class OutputFile {
 public:
   virtual ~OutputFile() {}
   virtual size_t write(const char* str) = 0;
-  virtual int printf(const char* format, ...) = 0;
   virtual int flush() = 0;
+  virtual int vprintf(const char* format, va_list va) = 0;
+  inline int printf(const char *format, ...) {
+    va_list va;
+    va_start(va, format);
+    int rv = vprintf(format, va);
+    va_end(va);
+    return rv;
+  }
   // Returns true if the output medium supports ANSI color codes.
   virtual bool supportsColor() = 0;
 };

src/WinConsoleFile.cc

@@ -72,18 +72,16 @@ size_t WinConsoleFile::write(const char* str)
   return written;
 }
 
-int WinConsoleFile::printf(const char* format, ...)
+int WinConsoleFile::vprintf(const char* format, va_list va)
 {
-  char buf[2048];
+  ssize_t r = _vscprintf(format, va);
-  va_list ap;
+  if (r <= 0) {
-  va_start(ap, format);
+    return 0;
-  int r = vsnprintf(buf, sizeof(buf), format, ap);
+  }
-  va_end(ap);
+  char *buf = new char[++r];
-  if(r == -1) {
+  r = vsnprintf(buf, r, format, va);
-    // MINGW32 vsnprintf returns -1 if output is truncated.
+  if (r < 0) {
-    r = strlen(buf);
+    delete [] buf;
-  } else if(r < 0) {
-    // Reachable?
     return 0;
   }
   DWORD written;
@@ -95,6 +93,7 @@ int WinConsoleFile::printf(const char* format, ...)
     WriteFile(GetStdHandle(stdHandle_),
               buf, r, &written, 0);
   }
+  delete [] buf;
   return written;
 }
 

src/WinConsoleFile.h

@@ -45,7 +45,7 @@ public:
   WinConsoleFile(DWORD stdHandle);
   virtual ~WinConsoleFile();
   virtual size_t write(const char* str);
-  virtual int printf(const char* format, ...);
+  virtual int vprintf(const char* format, va_list va);
   virtual int flush();
   virtual bool supportsColor();
 private: