https://codereview.chromium.org/99573002/diff/20001/src/promise.js
File src/promise.js (right):
https://codereview.chromium.org/99573002/diff/20001/src/promise.js#newcode63
src/promise.js:63: function Promise(resolver) {
On 2013/12/11 04:46:31, yhirano wrote:
On 2013/12/10 10:11:15, rossberg wrote:
> On 2013/12/09 02:31:36, yhirano wrote:
> > I'm still not sure.
> > The spec has not been changed at this point, has it?
>
> No, but the spec requires that exceptions from the function itself
are
delayed,
> so this CL makes it "more conforming" at least. An extra type check
would be
> needed to implement the synchronous type error.
Then can you write a comment?
Changed to throw
https://codereview.chromium.org/99573002/diff/20001/src/promise.js#newcode303
src/promise.js:303: "one", PromiseOne,
On 2013/12/10 14:21:24, arv wrote:
This should be race
Done.
https://codereview.chromium.org/99573002/
--
--
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/groups/opt_out.
