Code Review for jdk

Prepared by:jlaskey on Tue Jul 5 12:21:47 ADT 2016
Workspace:/Volumes/Elephant/Projects/jdk9-dev/jdk
Compare against: ssh://hg.openjdk.java.net/jdk9/dev/jdk
Compare against version:14945
Summary of changes: 5333 lines changed: 189 ins; 5140 del; 4 mod; 240 unchg
Changeset: jdk.changeset
Legend: Modified file
Deleted file
New file

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/jdk.jlink/share/classes/jdk/tools/jlink/resources/plugins.properties

rev 14946 : 8160829: Remove ASMPool support from jlink
Reviewed-by: sundar, psandoz, forax
5 lines changed: 0 ins; 2 del; 3 mod; 195 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/jdk.jlink/share/classes/module-info.java

rev 14946 : 8160829: Remove ASMPool support from jlink
Reviewed-by: sundar, psandoz, forax
3 lines changed: 1 ins; 1 del; 1 mod; 45 unchg

------ ------ ------ ------ --- New ----- Raw src/jdk.jlink/share/classes/jdk/tools/jlink/internal/plugins/ClassForNamePlugin.java

rev 14946 : 8160829: Remove ASMPool support from jlink
Reviewed-by: sundar, psandoz, forax
188 lines changed: 188 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ Old --- ----- --- src/jdk.jlink/share/classes/jdk/tools/jlink/internal/plugins/OptimizationPlugin.java

rev 14946 : 8160829: Remove ASMPool support from jlink
Reviewed-by: sundar, psandoz, forax
285 lines changed: 0 ins; 285 del; 0 mod; 0 unchg

------ ------ ------ ------ Old --- ----- --- src/jdk.jlink/share/classes/jdk/tools/jlink/internal/plugins/asm/AsmGlobalPool.java

rev 14946 : 8160829: Remove ASMPool support from jlink
Reviewed-by: sundar, psandoz, forax
54 lines changed: 0 ins; 54 del; 0 mod; 0 unchg

------ ------ ------ ------ Old --- ----- --- src/jdk.jlink/share/classes/jdk/tools/jlink/internal/plugins/asm/AsmModulePool.java

rev 14946 : 8160829: Remove ASMPool support from jlink
Reviewed-by: sundar, psandoz, forax
90 lines changed: 0 ins; 90 del; 0 mod; 0 unchg

------ ------ ------ ------ Old --- ----- --- src/jdk.jlink/share/classes/jdk/tools/jlink/internal/plugins/asm/AsmPlugin.java

rev 14946 : 8160829: Remove ASMPool support from jlink
Reviewed-by: sundar, psandoz, forax
68 lines changed: 0 ins; 68 del; 0 mod; 0 unchg

------ ------ ------ ------ Old --- ----- --- src/jdk.jlink/share/classes/jdk/tools/jlink/internal/plugins/asm/AsmPool.java

rev 14946 : 8160829: Remove ASMPool support from jlink
Reviewed-by: sundar, psandoz, forax
314 lines changed: 0 ins; 314 del; 0 mod; 0 unchg

------ ------ ------ ------ Old --- ----- --- src/jdk.jlink/share/classes/jdk/tools/jlink/internal/plugins/asm/AsmPoolImpl.java

rev 14946 : 8160829: Remove ASMPool support from jlink
Reviewed-by: sundar, psandoz, forax
698 lines changed: 0 ins; 698 del; 0 mod; 0 unchg

------ ------ ------ ------ Old --- ----- --- src/jdk.jlink/share/classes/jdk/tools/jlink/internal/plugins/asm/AsmPools.java

rev 14946 : 8160829: Remove ASMPool support from jlink
Reviewed-by: sundar, psandoz, forax
497 lines changed: 0 ins; 497 del; 0 mod; 0 unchg

------ ------ ------ ------ Old --- ----- --- src/jdk.jlink/share/classes/jdk/tools/jlink/internal/plugins/optim/ControlFlow.java

rev 14946 : 8160829: Remove ASMPool support from jlink
Reviewed-by: sundar, psandoz, forax
516 lines changed: 0 ins; 516 del; 0 mod; 0 unchg

------ ------ ------ ------ Old --- ----- --- src/jdk.jlink/share/classes/jdk/tools/jlink/internal/plugins/optim/ForNameFolding.java

rev 14946 : 8160829: Remove ASMPool support from jlink
Reviewed-by: sundar, psandoz, forax
111 lines changed: 0 ins; 111 del; 0 mod; 0 unchg

------ ------ ------ ------ Old --- ----- --- src/jdk.jlink/share/classes/jdk/tools/jlink/internal/plugins/optim/ReflectionOptimizer.java

rev 14946 : 8160829: Remove ASMPool support from jlink
Reviewed-by: sundar, psandoz, forax
226 lines changed: 0 ins; 226 del; 0 mod; 0 unchg

------ ------ ------ ------ Old --- ----- --- src/jdk.jlink/share/classes/jdk/tools/jlink/internal/plugins/optim/Utils.java

rev 14946 : 8160829: Remove ASMPool support from jlink
Reviewed-by: sundar, psandoz, forax
97 lines changed: 0 ins; 97 del; 0 mod; 0 unchg

------ ------ ------ ------ Old --- ----- --- test/tools/jlink/JLinkOptimTest.java

rev 14946 : 8160829: Remove ASMPool support from jlink
Reviewed-by: sundar, psandoz, forax
354 lines changed: 0 ins; 354 del; 0 mod; 0 unchg

------ ------ ------ ------ Old --- ----- --- test/tools/jlink/asmplugin/AddForgetResourcesTest.java

rev 14946 : 8160829: Remove ASMPool support from jlink
Reviewed-by: sundar, psandoz, forax
509 lines changed: 0 ins; 509 del; 0 mod; 0 unchg

------ ------ ------ ------ Old --- ----- --- test/tools/jlink/asmplugin/AsmPluginTestBase.java

rev 14946 : 8160829: Remove ASMPool support from jlink
Reviewed-by: sundar, psandoz, forax
249 lines changed: 0 ins; 249 del; 0 mod; 0 unchg

------ ------ ------ ------ Old --- ----- --- test/tools/jlink/asmplugin/BasicTest.java

rev 14946 : 8160829: Remove ASMPool support from jlink
Reviewed-by: sundar, psandoz, forax
177 lines changed: 0 ins; 177 del; 0 mod; 0 unchg

------ ------ ------ ------ Old --- ----- --- test/tools/jlink/asmplugin/IdentityPluginTest.java

rev 14946 : 8160829: Remove ASMPool support from jlink
Reviewed-by: sundar, psandoz, forax
121 lines changed: 0 ins; 121 del; 0 mod; 0 unchg

------ ------ ------ ------ Old --- ----- --- test/tools/jlink/asmplugin/NegativeTest.java

rev 14946 : 8160829: Remove ASMPool support from jlink
Reviewed-by: sundar, psandoz, forax
172 lines changed: 0 ins; 172 del; 0 mod; 0 unchg

------ ------ ------ ------ Old --- ----- --- test/tools/jlink/asmplugin/PackageMappingTest.java

rev 14946 : 8160829: Remove ASMPool support from jlink
Reviewed-by: sundar, psandoz, forax
197 lines changed: 0 ins; 197 del; 0 mod; 0 unchg

------ ------ ------ ------ Old --- ----- --- test/tools/jlink/asmplugin/SortingTest.java

rev 14946 : 8160829: Remove ASMPool support from jlink
Reviewed-by: sundar, psandoz, forax
183 lines changed: 0 ins; 183 del; 0 mod; 0 unchg

------ ------ ------ ------ Old --- ----- --- test/tools/jlink/asmplugin/VisitorTest.java

rev 14946 : 8160829: Remove ASMPool support from jlink
Reviewed-by: sundar, psandoz, forax
219 lines changed: 0 ins; 219 del; 0 mod; 0 unchg

This code review page was prepared using /Volumes/Elephant/Users/jlaskey/local/bin/webrev (vers 25.16-hg+openjdk.java.net).