Code Review for open

Prepared by:iklam on Wed Jul 22 12:00:48 PDT 2020
Workspace:/jdk2/gil/open
Compare against: ssh://hg.openjdk.java.net/jdk/jdk
Compare against version:4d36e29a5410
Summary of changes: 1261 lines changed: 1175 ins; 8 del; 78 mod; 44884 unchg
Patch of changes: open.patch
Legend: Modified file
Deleted file
New file

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/classfile/classLoader.cpp

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

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/classfile/classLoaderData.cpp

143 lines changed: 143 ins; 0 del; 0 mod; 970 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/classfile/classLoaderData.hpp

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

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/classfile/javaClasses.cpp

14 lines changed: 5 ins; 1 del; 8 mod; 4937 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/classfile/javaClasses.hpp

7 lines changed: 7 ins; 0 del; 0 mod; 1715 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/classfile/moduleEntry.cpp

189 lines changed: 186 ins; 0 del; 3 mod; 538 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/classfile/moduleEntry.hpp

26 lines changed: 24 ins; 0 del; 2 mod; 273 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/classfile/packageEntry.cpp

113 lines changed: 112 ins; 0 del; 1 mod; 346 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/classfile/packageEntry.hpp

14 lines changed: 14 ins; 0 del; 0 mod; 300 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/classfile/systemDictionary.cpp

11 lines changed: 11 ins; 0 del; 0 mod; 3048 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/classfile/systemDictionary.hpp

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

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/classfile/vmSymbols.hpp

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

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/memory/filemap.cpp

7 lines changed: 6 ins; 0 del; 1 mod; 2290 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/memory/filemap.hpp

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

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/memory/heapShared.cpp

99 lines changed: 85 ins; 0 del; 14 mod; 1084 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/memory/heapShared.hpp

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

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/memory/metaspaceShared.cpp

54 lines changed: 48 ins; 5 del; 1 mod; 2706 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/memory/metaspaceShared.hpp

18 lines changed: 14 ins; 2 del; 2 mod; 400 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/oops/instanceKlass.cpp

2 lines changed: 1 ins; 0 del; 1 mod; 4187 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/oops/instanceKlass.hpp

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

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/hotspot/share/runtime/arguments.cpp

9 lines changed: 7 ins; 0 del; 2 mod; 4391 unchg

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

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

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

56 lines changed: 52 ins; 0 del; 4 mod; 1674 unchg

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

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

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/java.base/share/classes/java/security/SecureClassLoader.java

6 lines changed: 5 ins; 0 del; 1 mod; 271 unchg

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

6 lines changed: 6 ins; 0 del; 0 mod; 352 unchg

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

17 lines changed: 16 ins; 0 del; 1 mod; 343 unchg

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

23 lines changed: 20 ins; 0 del; 3 mod; 1042 unchg

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

55 lines changed: 45 ins; 0 del; 10 mod; 202 unchg

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

124 lines changed: 111 ins; 0 del; 13 mod; 1063 unchg

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

7 lines changed: 7 ins; 0 del; 0 mod; 176 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/hotspot/jtreg/runtime/NMT/CheckForProperDetailStackTrace.java

22 lines changed: 21 ins; 0 del; 1 mod; 151 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/hotspot/jtreg/runtime/cds/appcds/SpecifySysLoaderProp.java

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

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/hotspot/jtreg/runtime/cds/appcds/jigsaw/modulepath/OptimizeModuleHandlingTest.java

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

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/hotspot/jtreg/runtime/logging/ModulesTest.java

9 lines changed: 5 ins; 0 del; 4 mod; 72 unchg

------ ------ ------ ------ --- New Patch Raw test/hotspot/jtreg/runtime/NMT/src/jdk.test/module-info.java

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

------ ------ ------ ------ --- New Patch Raw test/hotspot/jtreg/runtime/NMT/src/jdk.test/test/Main.java

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

------ ------ ------ ------ --- New Patch Raw test/hotspot/jtreg/runtime/cds/ServiceLoaderTest.java

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

This code review page was prepared using /home/iklam/jdk/proj/ioisvn/scripts/webrev.ksh (vers 25.16-hg+openjdk.java.net).