- rolled in some CString changes Harri Porten did on the KDE side
authordarin <darin@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 27 Oct 2003 04:21:15 +0000 (04:21 +0000)
committerdarin <darin@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 27 Oct 2003 04:21:15 +0000 (04:21 +0000)
        * kjs/ustring.cpp:
        (KJS::CString::CString): Use memcpy instead of strcpy for speed. Fix an off by one error
        in the copy constructor.
        (KJS::CString::operator=): Use memcpy instead of strcpy for speed.

        * JavaScriptCorePrefix.h: Add a definition of NULL here that matches the one in Merlot.
        This makes us see warnings that otherwise would be Merlot-only warnings.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@5264 268f45cc-cd09-0410-ab3c-d52691b4dbfc

JavaScriptCore/ChangeLog
JavaScriptCore/JavaScriptCorePrefix.h
JavaScriptCore/kjs/ustring.cpp

index bacc0f2..b0a7b14 100644 (file)
@@ -1 +1,13 @@
+2003-10-26  Darin Adler  <darin@apple.com>
+
+        - rolled in some CString changes Harri Porten did on the KDE side
+
+        * kjs/ustring.cpp:
+        (KJS::CString::CString): Use memcpy instead of strcpy for speed. Fix an off by one error
+        in the copy constructor.
+        (KJS::CString::operator=): Use memcpy instead of strcpy for speed.
+
+        * JavaScriptCorePrefix.h: Add a definition of NULL here that matches the one in Merlot.
+        This makes us see warnings that otherwise would be Merlot-only warnings.
+
 == Rolled over to ChangeLog-2003-10-25 ==
index 703a512..4c593a5 100644 (file)
@@ -1,5 +1,8 @@
-#include <config.h>
+#ifdef __cplusplus
+#define NULL __null
+#endif
 
+#include <config.h>
 
 #include <assert.h>
 #include <ctype.h>
index 1922249..8152053 100644 (file)
@@ -50,7 +50,7 @@ CString::CString(const char *c)
 {
   length = strlen(c);
   data = new char[length+1];
-  strcpy(data, c);
+  memcpy(data, c, length + 1);
 }
 
 CString::CString(const char *c, int len)
@@ -65,7 +65,7 @@ CString::CString(const CString &b)
 {
   length = b.length;
   data = new char[length+1];
-  memcpy(data, b.data, length);
+  memcpy(data, b.data, length + 1);
 }
 
 CString::~CString()
@@ -96,7 +96,7 @@ CString &CString::operator=(const char *c)
     delete [] data;
   length = strlen(c);
   data = new char[length+1];
-  strcpy(data, c);
+  memcpy(data, c, length + 1);
 
   return *this;
 }