Commends addressed, landing.
https://codereview.chromium.org/201873005/diff/10001/src/typedarray.js
File src/typedarray.js (right):
https://codereview.chromium.org/201873005/diff/10001/src/typedarray.js#newcode409
src/typedarray.js:409: ToPositiveDataViewOffset(offset),
On 2014/03/17 17:29:05, Jakob wrote:
Is this safe?
Looks safe to me.
https://codereview.chromium.org/201873005/diff/10001/test/mjsunit/regress/regress-353004.js
File test/mjsunit/regress/regress-353004.js (right):
https://codereview.chromium.org/201873005/diff/10001/test/mjsunit/regress/regress-353004.js#newcode2
test/mjsunit/regress/regress-353004.js:2: // Redistribution and use in
source and binary forms, with or without
On 2014/03/17 17:29:05, Jakob wrote:
nit: you can use the new, short licence header.
Done.
https://codereview.chromium.org/201873005/diff/10001/test/mjsunit/regress/regress-353004.js#newcode30
test/mjsunit/regress/regress-353004.js:30: buffer1 = new ArrayBuffer(100
* 1024 * 1024);
On 2014/03/17 17:29:05, Jakob wrote:
Wouldn't 100K be enough for each bufferX? I fear that working with big
arrays
might lead to OOM crashes on memory-constrained devices/buildbots.
Done.
https://codereview.chromium.org/201873005/diff/10001/test/mjsunit/regress/regress-353004.js#newcode32
test/mjsunit/regress/regress-353004.js:32: array1 = new
Uint8Array(buffer1, {valueOf : function() {
On 2014/03/17 17:29:05, Jakob wrote:
nit: how about some "var" keywords, here and for every other variable
in this
test?
Done.
https://codereview.chromium.org/201873005/diff/10001/test/mjsunit/regress/regress-353004.js#newcode38
test/mjsunit/regress/regress-353004.js:38: for (i = 0; i <
array1.length; i++) {
On 2014/03/17 17:29:05, Jakob wrote:
This loop is pointless; if the assertEquals right above passed, it
runs for zero
iterations.
Done.
https://codereview.chromium.org/201873005/diff/10001/test/mjsunit/regress/regress-353004.js#newcode59
test/mjsunit/regress/regress-353004.js:59: for (i = 0; i <
dataView1.byteLength; i++) {
On 2014/03/17 17:29:05, Jakob wrote:
another pointless loop
Done.
https://codereview.chromium.org/201873005/diff/10001/test/mjsunit/regress/regress-353004.js#newcode91
test/mjsunit/regress/regress-353004.js:91: %ArrayBufferNeuter(buffer9);
On 2014/03/17 17:29:05, Jakob wrote:
nit: funky indentation
Done.
https://codereview.chromium.org/201873005/
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/d/optout.
