Code Review for openjdk-head

Prepared by:Severin Gehwolf on Mon Feb 11 19:00:47 CET 2019
Workspace:/disk/openjdk/upstream-sources/openjdk-head
Compare against: https://hg.openjdk.java.net/jdk/jdk
Compare against version:3f054fd85646
Summary of changes: 988 lines changed: 984 ins; 0 del; 4 mod; 350 unchg
Patch of changes: openjdk-head.patch
Legend: Modified file
Deleted file
New file

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw make/common/TestFilesCompilation.gmk

@  rev 53689 : 8214796: Create a jlink plugin for stripping debug info symbols from native libraries
|  Reviewed-by: alanb, mchung, erikj
~
1 line changed: 0 ins; 0 del; 1 mod; 114 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw make/gensrc/Gensrc-jdk.jlink.gmk

@  rev 53689 : 8214796: Create a jlink plugin for stripping debug info symbols from native libraries
|  Reviewed-by: alanb, mchung, erikj
~
o  rev 53378 : 8214796: V3: Create a jlink plugin for stripping debug info symbols from native libraries
|  Reviewed-by: alanb, mandy
~
o  rev 53375 : 8214796: Create a jlink plugin for stripping debug info symbols from native libraries
|  Reviewed-by: alanb, mandy
~
5 lines changed: 3 ins; 0 del; 2 mod; 52 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw make/test/JtregNativeJdk.gmk

@  rev 53689 : 8214796: Create a jlink plugin for stripping debug info symbols from native libraries
|  Reviewed-by: alanb, mchung, erikj
~
8 lines changed: 8 ins; 0 del; 0 mod; 117 unchg

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

@  rev 53689 : 8214796: Create a jlink plugin for stripping debug info symbols from native libraries
|  Reviewed-by: alanb, mchung, erikj
~
o  rev 53377 : Move plugin strings into separate resource files
|  Reviewed-by: duke
~
5 lines changed: 4 ins; 0 del; 1 mod; 67 unchg

------ ------ ------ ------ --- New Patch Raw src/jdk.jlink/linux/classes/jdk/tools/jlink/internal/plugins/StripNativeDebugSymbolsPlugin.java

@  rev 53689 : 8214796: Create a jlink plugin for stripping debug info symbols from native libraries
|  Reviewed-by: alanb, mchung, erikj
~
311 lines changed: 311 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw src/jdk.jlink/linux/classes/jdk/tools/jlink/resources/strip_native_debug_symbols_plugin.properties

@  rev 53689 : 8214796: Create a jlink plugin for stripping debug info symbols from native libraries
|  Reviewed-by: alanb, mchung, erikj
~
38 lines changed: 38 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw src/jdk.jlink/linux/classes/module-info.java.extra

@  rev 53689 : 8214796: Create a jlink plugin for stripping debug info symbols from native libraries
|  Reviewed-by: alanb, mchung, erikj
~
27 lines changed: 27 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw test/jdk/tools/jlink/plugins/strip-native-debug-symbols/StripNativeDebugSymbolsPluginTest.java

@  rev 53689 : 8214796: Create a jlink plugin for stripping debug info symbols from native libraries
|  Reviewed-by: alanb, mchung, erikj
~
471 lines changed: 471 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw test/jdk/tools/jlink/plugins/strip-native-debug-symbols/src/fib/FibJNI.java

@  rev 53689 : 8214796: Create a jlink plugin for stripping debug info symbols from native libraries
|  Reviewed-by: alanb, mchung, erikj
~
63 lines changed: 63 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw test/jdk/tools/jlink/plugins/strip-native-debug-symbols/src/libFib.c

@  rev 53689 : 8214796: Create a jlink plugin for stripping debug info symbols from native libraries
|  Reviewed-by: alanb, mchung, erikj
~
59 lines changed: 59 ins; 0 del; 0 mod; 0 unchg

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