Code Review for nashorn

Prepared by:hannesw on Fri Nov 4 21:48:17 CET 2016
Workspace:/Users/hannes/Work/jdk9-dev/nashorn
Summary of changes: 507 lines changed: 505 ins; 0 del; 2 mod; 8798 unchg
Patch of changes: nashorn.patch
Bug id: JDK-8156619 : Unimplemented ES6 features should result in clear Error being thrown
Legend: Modified file
Deleted file
New file

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/codegen/AssignSymbols.java

1 line changed: 0 ins; 0 del; 1 mod; 1033 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/codegen/Lower.java

89 lines changed: 89 ins; 0 del; 0 mod; 769 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/ir/visitor/NodeOperatorVisitor.java

1 line changed: 0 ins; 0 del; 1 mod; 1217 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/parser/Parser.java

12 lines changed: 12 ins; 0 del; 0 mod; 5555 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/jdk.scripting.nashorn/share/classes/jdk/nashorn/internal/runtime/resources/Messages.properties

10 lines changed: 10 ins; 0 del; 0 mod; 224 unchg

------ ------ ------ ------ --- New Patch Raw test/script/basic/es6/class.js

36 lines changed: 36 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw test/script/basic/es6/class.js.EXPECTED

3 lines changed: 3 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw test/script/basic/es6/destructuring.js

65 lines changed: 65 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw test/script/basic/es6/destructuring.js.EXPECTED

72 lines changed: 72 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw test/script/basic/es6/generator.js

42 lines changed: 42 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw test/script/basic/es6/generator.js.EXPECTED

9 lines changed: 9 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw test/script/basic/es6/restparam.js

46 lines changed: 46 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw test/script/basic/es6/restparam.js.EXPECTED

15 lines changed: 15 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw test/script/basic/es6/spread.js

43 lines changed: 43 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw test/script/basic/es6/spread.js.EXPECTED

12 lines changed: 12 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw test/script/basic/es6/super.js

42 lines changed: 42 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw test/script/basic/es6/super.js.EXPECTED

9 lines changed: 9 ins; 0 del; 0 mod; 0 unchg

This code review page was prepared using /Users/hannes/webrev.ksh (vers 25.16-hg+openjdk.java.net).