Minor cleanup: the smartpointer class should use DeleteArray to

deallocate.

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@137 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

diff --git a/src/objects.cc b/src/objects.cc
index 87e07d9..2be4ccd 100644 (file)
--- a/src/objects.cc
+++ b/src/objects.cc
@@ -2968,9 +2968,9 @@ SmartPointer<char> String::ToCString(AllowNullsFlag allow_nulls,
 }
 
 
-SmartPointer < char >String::ToCString(AllowNullsFlag allow_nulls,
-                                       RobustnessFlag robust_flag,
-                                       int* length_return) {
+SmartPointer<char> String::ToCString(AllowNullsFlag allow_nulls,
+                                     RobustnessFlag robust_flag,
+                                     int* length_return) {
   return ToCString(allow_nulls, robust_flag, 0, -1, length_return);
 }
 

diff --git a/src/smart-pointer.h b/src/smart-pointer.h
index 3fe27d4..7ff35c2 100644 (file)
--- a/src/smart-pointer.h
+++ b/src/smart-pointer.h
@@ -41,9 +41,9 @@ class SmartPointer {
 
 
   // When the destructor of the scoped pointer is executed the plain pointer
-  // is deleted using delete[].  This implies that you must allocate with
-  // new[...], not new(...).
-  inline ~SmartPointer() { if (p) delete [] p; }
+  // is deleted using DeleteArray.  This implies that you must allocate with
+  // NewArray.
+  inline ~SmartPointer() { if (p) DeleteArray(p); }
 
 
   // Copy constructor removes the pointer from the original to avoid double