Comment #3 on issue 4228 by [email protected]: ToString not called for
Number wrappers in JSON.stringify
https://code.google.com/p/v8/issues/detail?id=4228#c3
The following revision refers to this bug:
https://chromium.googlesource.com/v8/v8.git/+/6e6a1c8aee16cffdcbbf54f09b73b2b79b2ecedf
commit 6e6a1c8aee16cffdcbbf54f09b73b2b79b2ecedf
Author: arv <[email protected]>
Date: Wed Jun 24 22:17:52 2015
JSON.stringify should use toString of replacer and not valueOf
If the replacer array contains a number wrapper we should use the
toString result and not valueOf.
BUG=v8:4228
LOG=N
R=adamk
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_chromium_rel_ng;tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1207013002
Cr-Commit-Position: refs/heads/master@{#29270}
[modify]
http://crrev.com/6e6a1c8aee16cffdcbbf54f09b73b2b79b2ecedf/src/json.js
[add]
http://crrev.com/6e6a1c8aee16cffdcbbf54f09b73b2b79b2ecedf/test/mjsunit/json-replacer-number-wrapper-tostring.js
--
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.
