Revision: 3915
Author: [email protected]
Date: Fri Feb 19 04:24:05 2010
Log: Merge revision 3913 and 3914 to trunk to fix assertion failures.
Review URL: http://codereview.chromium.org/650027
http://code.google.com/p/v8/source/detail?r=3915

Modified:
 /trunk/src/objects.cc
 /trunk/src/runtime.cc
 /trunk/src/version.cc

=======================================
--- /trunk/src/objects.cc       Fri Feb 19 00:53:10 2010
+++ /trunk/src/objects.cc       Fri Feb 19 04:24:05 2010
@@ -261,6 +261,7 @@
                                                   name,
                                                   attributes);
         }
+        break;
       }
       default:
         UNREACHABLE();
=======================================
--- /trunk/src/runtime.cc       Fri Feb 19 00:53:10 2010
+++ /trunk/src/runtime.cc       Fri Feb 19 04:24:05 2010
@@ -2960,8 +2960,6 @@
   // Check if the given key is an array index.
   uint32_t index;
   if (Array::IndexFromObject(*key, &index)) {
-    ASSERT(attr == NONE);
-
// In Firefox/SpiderMonkey, Safari and Opera you can access the characters
     // of a string using [] notation.  We need to support this too in
     // JavaScript.
@@ -2981,7 +2979,6 @@
   if (key->IsString()) {
     Handle<Object> result;
     if (Handle<String>::cast(key)->AsArrayIndex(&index)) {
-      ASSERT(attr == NONE);
       result = SetElement(js_object, index, value);
     } else {
       Handle<String> key_string = Handle<String>::cast(key);
@@ -2999,7 +2996,6 @@
   Handle<String> name = Handle<String>::cast(converted);

   if (name->AsArrayIndex(&index)) {
-    ASSERT(attr == NONE);
     return js_object->SetElement(index, *value);
   } else {
     return js_object->SetProperty(*name, *value, attr);
@@ -3016,8 +3012,6 @@
   // Check if the given key is an array index.
   uint32_t index;
   if (Array::IndexFromObject(*key, &index)) {
-    ASSERT(attr == NONE);
-
// In Firefox/SpiderMonkey, Safari and Opera you can access the characters
     // of a string using [] notation.  We need to support this too in
     // JavaScript.
@@ -3034,7 +3028,6 @@

   if (key->IsString()) {
     if (Handle<String>::cast(key)->AsArrayIndex(&index)) {
-      ASSERT(attr == NONE);
       return js_object->SetElement(index, *value);
     } else {
       Handle<String> key_string = Handle<String>::cast(key);
@@ -3052,7 +3045,6 @@
   Handle<String> name = Handle<String>::cast(converted);

   if (name->AsArrayIndex(&index)) {
-    ASSERT(attr == NONE);
     return js_object->SetElement(index, *value);
   } else {
return js_object->IgnoreAttributesAndSetLocalProperty(*name, *value, attr);
=======================================
--- /trunk/src/version.cc       Fri Feb 19 00:53:10 2010
+++ /trunk/src/version.cc       Fri Feb 19 04:24:05 2010
@@ -35,7 +35,7 @@
 #define MAJOR_VERSION     2
 #define MINOR_VERSION     1
 #define BUILD_NUMBER      1
-#define PATCH_LEVEL       0
+#define PATCH_LEVEL       1
 #define CANDIDATE_VERSION false

 // Define SONAME to have the SCons build the put a specific SONAME into the

--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to