Code Review for openjdk-head

Prepared by:Severin Gehwolf on Thu Feb 7 17:49:21 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: 980 lines changed: 977 ins; 0 del; 3 mod; 351 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
~
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
~
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
~
12 lines changed: 11 ins; 0 del; 1 mod; 53 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
~
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
~
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/unix/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
~
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
~
313 lines changed: 313 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw src/jdk.jlink/unix/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
~
o  rev 53378 : 8214796: V3: Create a jlink plugin for stripping debug info symbols from native libraries
|  Reviewed-by: alanb, mandy
~
38 lines changed: 38 ins; 0 del; 0 mod; 0 unchg

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

@  rev 53689 : 8214796: Create a jlink plugin for stripping debug info symbols from native libraries
|  Reviewed-by: alanb, mchung
~
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
~
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
~
456 lines changed: 456 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
~
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
~
57 lines changed: 57 ins; 0 del; 0 mod; 0 unchg

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