https://codereview.chromium.org/1218553004/diff/20001/test/mjsunit/d8-worker-sharedarraybuffer.js
File test/mjsunit/d8-worker-sharedarraybuffer.js (right):
https://codereview.chromium.org/1218553004/diff/20001/test/mjsunit/d8-worker-sharedarraybuffer.js#newcode42
test/mjsunit/d8-worker-sharedarraybuffer.js:42: // Atomically update
ta[0]\n"
On 2015/07/01 at 07:35:43, jarin wrote:
Nit: remove the extra stuff at the end of the line.
Done.
https://codereview.chromium.org/1218553004/diff/20001/test/mjsunit/d8-worker.js
File test/mjsunit/d8-worker.js (right):
https://codereview.chromium.org/1218553004/diff/20001/test/mjsunit/d8-worker.js#newcode33
test/mjsunit/d8-worker.js:33: "postMessage('Starting worker');\n" +
On 2015/07/01 at 07:35:44, jarin wrote:
You might consider using the new shiny template strings for multi-line
strings. Example:
`a
b`
See
https://developer.mozilla.org/en/docs/Web/JavaScript/Reference/template_strings.
Ah, I didn't know about this. Thanks!
https://codereview.chromium.org/1218553004/
--
--
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.
