Comment #15 on issue 4211 by [email protected]: Wrong precedence for arrow function without parameters
https://code.google.com/p/v8/issues/detail?id=4211#c15
The following revision refers to this bug:https://chromium.googlesource.com/v8/v8.git/+/cc97e524a151c44746d4c569a828373a282797b2
commit cc97e524a151c44746d4c569a828373a282797b2 Author: yangguo <[email protected]> Date: Mon Aug 24 06:56:56 2015Revert of Parse arrow functions at proper precedence level (patchset #2 id:60001 of https://codereview.chromium.org/1286383005/ )
Reason for revert: Breaks layout test. Please change test expectation on blink first.--- /mnt/data/b/build/slave/V8-Blink_Linux_64/build/layout-test-results/inspector/sources/debugger-pause/debugger-pause-in-internal-expected.txt +++ /mnt/data/b/build/slave/V8-Blink_Linux_64/build/layout-test-results/inspector/sources/debugger-pause/debugger-pause-in-internal-actual.txt
@@ -1,4 +1,4 @@-CONSOLE ERROR: line 9: Uncaught SyntaxError: Expected () to start arrow function, but got '}' instead of '=>'
+CONSOLE ERROR: line 9: Uncaught SyntaxError: Unexpected token ) Tests that pause on exception in internal script does not crash. Script source was shown. Original issue's description:
Parse arrow functions at proper precedence level
BUG=v8:4211 LOG=Y [email protected]
Committed: https://crrev.com/9271b0ccf9ddb217deb1f0b9ef9b59b64dc40214 Cr-Commit-Position: refs/heads/master@{#30298}
[email protected],[email protected],[email protected],[email protected],[email protected] NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=v8:4211 Review URL: https://codereview.chromium.org/1315503002 Cr-Commit-Position: refs/heads/master@{#30318}[modify] http://crrev.com/cc97e524a151c44746d4c569a828373a282797b2/src/ast-literal-reindexer.cc [modify] http://crrev.com/cc97e524a151c44746d4c569a828373a282797b2/src/ast-numbering.cc
[modify] http://crrev.com/cc97e524a151c44746d4c569a828373a282797b2/src/ast.h[modify] http://crrev.com/cc97e524a151c44746d4c569a828373a282797b2/src/compiler/ast-graph-builder.cc [modify] http://crrev.com/cc97e524a151c44746d4c569a828373a282797b2/src/compiler/ast-loop-assignment-analyzer.cc [modify] http://crrev.com/cc97e524a151c44746d4c569a828373a282797b2/src/full-codegen/full-codegen.cc [modify] http://crrev.com/cc97e524a151c44746d4c569a828373a282797b2/src/hydrogen.cc [modify] http://crrev.com/cc97e524a151c44746d4c569a828373a282797b2/src/interpreter/bytecode-generator.cc [modify] http://crrev.com/cc97e524a151c44746d4c569a828373a282797b2/src/messages.h [modify] http://crrev.com/cc97e524a151c44746d4c569a828373a282797b2/src/parser.cc [modify] http://crrev.com/cc97e524a151c44746d4c569a828373a282797b2/src/pattern-rewriter.cc [modify] http://crrev.com/cc97e524a151c44746d4c569a828373a282797b2/src/preparser.h [modify] http://crrev.com/cc97e524a151c44746d4c569a828373a282797b2/src/prettyprinter.cc [modify] http://crrev.com/cc97e524a151c44746d4c569a828373a282797b2/src/typing.cc [modify] http://crrev.com/cc97e524a151c44746d4c569a828373a282797b2/test/message/arrow-missing.out [delete] http://crrev.com/29110072403e03c4384d615628e767cbe58a2cb8/test/mjsunit/harmony/regress/regress-4211.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.
