Code Review for jdk

Prepared by:alanb on Thu Oct 27 12:14:11 IST 2016
Workspace:/Users/albatem/ws/jdk9-dev/jdk
Summary of changes: 1146 lines changed: 778 ins; 172 del; 196 mod; 10937 unchg
Patch of changes: jdk.patch
Legend: Modified file
Deleted file
New file

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/java.base/share/classes/java/lang/module/ModuleDescriptor.java

8 lines changed: 8 ins; 0 del; 0 mod; 2003 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/java.base/share/classes/java/lang/module/ModuleFinder.java

23 lines changed: 7 ins; 0 del; 16 mod; 369 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/java.base/share/classes/java/lang/module/ModulePath.java

149 lines changed: 74 ins; 23 del; 52 mod; 553 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/java.base/share/classes/java/lang/module/ModuleReader.java

43 lines changed: 34 ins; 5 del; 4 mod; 181 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/java.base/share/classes/java/lang/module/ModuleReferences.java

71 lines changed: 57 ins; 0 del; 14 mod; 372 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/java.base/share/classes/java/lang/module/SystemModuleFinder.java

104 lines changed: 100 ins; 3 del; 1 mod; 337 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/java.base/share/classes/jdk/internal/jmod/JmodFile.java

20 lines changed: 19 ins; 1 del; 0 mod; 211 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/java.base/share/classes/jdk/internal/loader/BuiltinClassLoader.java

5 lines changed: 5 ins; 0 del; 0 mod; 764 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/java.base/share/classes/jdk/internal/loader/Loader.java

5 lines changed: 5 ins; 0 del; 0 mod; 542 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/java.base/share/classes/jdk/internal/misc/JavaLangModuleAccess.java

10 lines changed: 8 ins; 0 del; 2 mod; 125 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/java.base/share/classes/jdk/internal/module/ModuleInfoWriter.java

9 lines changed: 8 ins; 0 del; 1 mod; 98 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/java.base/share/classes/jdk/internal/module/ModulePatcher.java

38 lines changed: 28 ins; 2 del; 8 mod; 565 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/jdk.jlink/share/classes/jdk/tools/jlink/internal/JlinkTask.java

36 lines changed: 23 ins; 6 del; 7 mod; 547 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/jdk.jlink/share/classes/jdk/tools/jlink/internal/TaskHelper.java

11 lines changed: 1 ins; 6 del; 4 mod; 746 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/jdk.jlink/share/classes/jdk/tools/jmod/JmodTask.java

5 lines changed: 0 ins; 4 del; 1 mod; 1424 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/java/lang/module/AutomaticModulesTest.java

108 lines changed: 83 ins; 9 del; 16 mod; 666 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/java/lang/module/ModuleFinderTest.java

118 lines changed: 116 ins; 0 del; 2 mod; 690 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/java/lang/module/ModuleReader/ModuleReaderTest.java

48 lines changed: 31 ins; 5 del; 12 mod; 320 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/java/util/ResourceBundle/modules/basic/src/mainbundles/jdk/test/resources/MyResourcesProvider.java

3 lines changed: 0 ins; 2 del; 1 mod; 79 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/tools/jmod/hashes/HashesTest.java

21 lines changed: 3 ins; 12 del; 6 mod; 199 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/java/lang/module/MultiReleaseJarTest.java (was test/java/lang/module/ModuleReader/MultiReleaseJarTest.java)

265 lines changed: 168 ins; 48 del; 49 mod; 119 unchg

------ ------ ------ ------ --- New Patch Raw test/java/util/ResourceBundle/modules/basic/src/asiabundles/jdk/test/resources/asia/MyResources_ja_JP.properties (was test/java/util/ResourceBundle/modules/basic/src/asiabundles/jdk/test/resources/MyResources_ja_JP.properties)

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

------ ------ ------ ------ Old --- Patch --- src/java.base/share/classes/jdk/internal/module/ConfigurableModuleFinder.java

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

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