Remove custom CharArray type

It has now been replaced, mostly by std::string, so remove the actual
type definition.

diff --git a/common/rfb/Hostname.h b/common/rfb/Hostname.h
index 8e57f31db17ba18787d469cdf6a46ed584245537..1971e343177c46e2ee4b72f068eba2a7a04c8023 100644 (file)
--- a/common/rfb/Hostname.h
+++ b/common/rfb/Hostname.h
@@ -22,6 +22,7 @@
 #include <assert.h>
 #include <ctype.h>
 #include <stdlib.h>
+#include <string.h>
 #include <rdr/Exception.h>
 #include <rfb/util.h>
 

diff --git a/common/rfb/PixelBuffer.cxx b/common/rfb/PixelBuffer.cxx
index dedae3b23a763e5ff5c4e03d2a67943662475cbc..28eea9a764e3f38e092297122760dbbd9eca239a 100644 (file)
--- a/common/rfb/PixelBuffer.cxx
+++ b/common/rfb/PixelBuffer.cxx
@@ -26,6 +26,8 @@
 #include <config.h>
 #endif
 
+#include <string.h>
+
 #include <rfb/Exception.h>
 #include <rfb/LogWriter.h>
 #include <rfb/PixelBuffer.h>

diff --git a/common/rfb/util.cxx b/common/rfb/util.cxx
index c5eeed92c3899927e6e872d971541db3415c18d8..0df9e1e3cfefc86b81263af6fdb31473cf59b89f 100644 (file)
--- a/common/rfb/util.cxx
+++ b/common/rfb/util.cxx
@@ -25,35 +25,13 @@
 #include <ctype.h>
 #include <stdarg.h>
 #include <stdio.h>
+#include <string.h>
 #include <sys/time.h>
 
 #include <rfb/util.h>
 
 namespace rfb {
 
-  void CharArray::format(const char *fmt, ...) {
-    va_list ap;
-    int len;
-
-    va_start(ap, fmt);
-    len = vsnprintf(NULL, 0, fmt, ap);
-    va_end(ap);
-
-    delete [] buf;
-
-    if (len < 0) {
-      buf = new char[1];
-      buf[0] = '\0';
-      return;
-    }
-
-    buf = new char[len+1];
-
-    va_start(ap, fmt);
-    vsnprintf(buf, len+1, fmt, ap);
-    va_end(ap);
-  }
-
   char* strDup(const char* s) {
     if (!s) return 0;
     int l = strlen(s);

diff --git a/common/rfb/util.h b/common/rfb/util.h
index b361aa118c3c531120f03c31131940ddfe0c8426..799e0756c1d70b7415f58f895b674fdb681a356d 100644 (file)
--- a/common/rfb/util.h
+++ b/common/rfb/util.h
@@ -25,7 +25,6 @@
 #define __RFB_UTIL_H__
 
 #include <limits.h>
-#include <string.h>
 #include <stdint.h>
 
 #include <string>
@@ -35,29 +34,6 @@ struct timeval;
 
 namespace rfb {
 
-  // -=- Class to handle cleanup of arrays of characters
-  class CharArray {
-  public:
-    CharArray() : buf(0) {}
-    CharArray(char* str) : buf(str) {} // note: assumes ownership
-    CharArray(size_t len) {
-      buf = new char[len]();
-      memset(buf, 0, len);
-    }
-    ~CharArray() {
-      delete [] buf;
-    }
-    void format(const char *fmt, ...)
-      __attribute__((__format__ (__printf__, 2, 3)));
-    // Get the buffer pointer & clear it (i.e. caller takes ownership)
-    char* takeBuf() {char* tmp = buf; buf = 0; return tmp;}
-    void replaceBuf(char* b) {delete [] buf; buf = b;}
-    char* buf;
-  private:
-    CharArray(const CharArray&);
-    CharArray& operator=(const CharArray&);
-  };
-
   char* strDup(const char* s);
   void strFree(char* s);
 

diff --git a/tests/unit/unicode.cxx b/tests/unit/unicode.cxx
index 32452ee3c8351b4d81a1ab8e1c16882f6aa22fcb..4618135d0528046b55b3971d5346536eb2e1f329 100644 (file)
--- a/tests/unit/unicode.cxx
+++ b/tests/unit/unicode.cxx
@@ -21,6 +21,7 @@
 #endif
 
 #include <stdio.h>
+#include <string.h>
 #include <wchar.h>
 
 #include <rfb/util.h>

diff --git a/unix/tx/TXWindow.cxx b/unix/tx/TXWindow.cxx
index c771a6235d39d7d997583e25585922c54f37ce96..a631fa6685eac8b7b57aec517009783df8789f52 100644 (file)
--- a/unix/tx/TXWindow.cxx
+++ b/unix/tx/TXWindow.cxx
@@ -28,6 +28,7 @@
 #include <list>
 #include <stdio.h>
 #include <stdlib.h>
+#include <string.h>
 #include <vector>
 #include <rfb/util.h>