Commit 23dbc85b authored by yangguo@chromium.org's avatar yangguo@chromium.org

Fix more MUST_USE_RESULT warnings.

R=dcarney@chromium.org

Review URL: https://codereview.chromium.org/228233002

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@20563 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
parent 1d138ecd
...@@ -779,7 +779,7 @@ TEST(JSArray) { ...@@ -779,7 +779,7 @@ TEST(JSArray) {
CHECK(array->HasFastSmiOrObjectElements()); CHECK(array->HasFastSmiOrObjectElements());
// array[length] = name. // array[length] = name.
JSReceiver::SetElement(array, 0, name, NONE, SLOPPY); JSReceiver::SetElement(array, 0, name, NONE, SLOPPY).Check();
CHECK_EQ(Smi::FromInt(1), array->length()); CHECK_EQ(Smi::FromInt(1), array->length());
CHECK_EQ(*i::Object::GetElement(isolate, array, 0), *name); CHECK_EQ(*i::Object::GetElement(isolate, array, 0), *name);
...@@ -794,7 +794,7 @@ TEST(JSArray) { ...@@ -794,7 +794,7 @@ TEST(JSArray) {
CHECK(array->HasDictionaryElements()); // Must be in slow mode. CHECK(array->HasDictionaryElements()); // Must be in slow mode.
// array[length] = name. // array[length] = name.
JSReceiver::SetElement(array, int_length, name, NONE, SLOPPY); JSReceiver::SetElement(array, int_length, name, NONE, SLOPPY).Check();
uint32_t new_int_length = 0; uint32_t new_int_length = 0;
CHECK(array->length()->ToArrayIndex(&new_int_length)); CHECK(array->length()->ToArrayIndex(&new_int_length));
CHECK_EQ(static_cast<double>(int_length), new_int_length - 1); CHECK_EQ(static_cast<double>(int_length), new_int_length - 1);
...@@ -824,8 +824,8 @@ TEST(JSObjectCopy) { ...@@ -824,8 +824,8 @@ TEST(JSObjectCopy) {
JSReceiver::SetProperty(obj, first, one, NONE, SLOPPY).Check(); JSReceiver::SetProperty(obj, first, one, NONE, SLOPPY).Check();
JSReceiver::SetProperty(obj, second, two, NONE, SLOPPY).Check(); JSReceiver::SetProperty(obj, second, two, NONE, SLOPPY).Check();
JSReceiver::SetElement(obj, 0, first, NONE, SLOPPY); JSReceiver::SetElement(obj, 0, first, NONE, SLOPPY).Check();
JSReceiver::SetElement(obj, 1, second, NONE, SLOPPY); JSReceiver::SetElement(obj, 1, second, NONE, SLOPPY).Check();
// Make the clone. // Make the clone.
Handle<JSObject> clone = JSObject::Copy(obj); Handle<JSObject> clone = JSObject::Copy(obj);
...@@ -843,8 +843,8 @@ TEST(JSObjectCopy) { ...@@ -843,8 +843,8 @@ TEST(JSObjectCopy) {
JSReceiver::SetProperty(clone, first, two, NONE, SLOPPY).Check(); JSReceiver::SetProperty(clone, first, two, NONE, SLOPPY).Check();
JSReceiver::SetProperty(clone, second, one, NONE, SLOPPY).Check(); JSReceiver::SetProperty(clone, second, one, NONE, SLOPPY).Check();
JSReceiver::SetElement(clone, 0, second, NONE, SLOPPY); JSReceiver::SetElement(clone, 0, second, NONE, SLOPPY).Check();
JSReceiver::SetElement(clone, 1, first, NONE, SLOPPY); JSReceiver::SetElement(clone, 1, first, NONE, SLOPPY).Check();
CHECK_EQ(*i::Object::GetElement(isolate, obj, 1), CHECK_EQ(*i::Object::GetElement(isolate, obj, 1),
*i::Object::GetElement(isolate, clone, 0)); *i::Object::GetElement(isolate, clone, 0));
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment