Comment #2 on issue 3241 by [email protected]: Object.prototype.toString.call(Promise.resolve()) should be "[object Promise]"
http://code.google.com/p/v8/issues/detail?id=3241

I'm interested in putting together a fix for this, because I'd like to get more acquainted with the VM's code.

My work so far involves updating the toString() algorithm to match section 19.1.3.6 of the most recent harmony draft, to take the @@toString symbol into account.

I'm not positive that this is the best (or most backwards-compatible) approach, or whether additional changes are needed to accommodate the algorithm change, and I'm not absolutely positive yet how to best manage symbols (but I'm hoping to learn this as I go). So I do know this isn't quite right, but I'd like to help get it there.

If anyone feels like offering a few minor hints at the correct way to go about implementing this (and the updated toString algorithm in general), I'm happy to pair or even just have a brief chat about it.

Anyways, the patch so far is at https://codereview.chromium.org/227413004, if anyone wants to help me clean that up a bit.

--
You received this message because this project is configured to send all issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--
--
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.

Reply via email to