Correct nits in d8 ArrayBuffer() implementation
authordanno@chromium.org <danno@chromium.org@ce2b1a6d-e550-0410-aec6-3dcde31c8c00>
Wed, 11 Jan 2012 15:10:56 +0000 (15:10 +0000)
committerdanno@chromium.org <danno@chromium.org@ce2b1a6d-e550-0410-aec6-3dcde31c8c00>
Wed, 11 Jan 2012 15:10:56 +0000 (15:10 +0000)
TBR=jkummerow@chromium.org
BUG=none
TEST=external-array.js

Review URL: http://codereview.chromium.org/9185006

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

src/d8.cc
test/mjsunit/external-array.js

index 927d536..97828a4 100644 (file)
--- a/src/d8.cc
+++ b/src/d8.cc
@@ -387,8 +387,8 @@ Handle<Value> Shell::CreateExternalArray(const Arguments& args,
     }
 
     if (offset > array_buffer_length) {
-        return ThrowException(
-            String::New("byteOffset must be less than ArrayBuffer length."));
+      return ThrowException(
+          String::New("byteOffset must be less than ArrayBuffer length."));
     }
 
     if (args.Length() == 2) {
index 7bc882f..72cfd85 100644 (file)
@@ -1,4 +1,4 @@
-// Copyright 2011 the V8 project authors. All rights reserved.
+// Copyright 2012 the V8 project authors. All rights reserved.
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
@@ -60,12 +60,6 @@ assertEquals(2, derived_uint32_2.length);
 var derived_uint32_3 = new Uint32Array(ab,4,1);
 assertEquals(1, derived_uint32_3.length);
 
-// Resulting array length of zero should fail.
-function abfunc2() {
-  new Uint32Array(ab,3);
-}
-assertThrows(abfunc2);
-
 // If a given byteOffset and length references an area beyond the end of the
 // ArrayBuffer an exception is raised.
 function abfunc3() {