Code Review for openjdk-head

Prepared by:Severin Gehwolf on Fri Feb 15 17:55:03 CET 2019
Workspace:/disk/openjdk/upstream-sources/openjdk-head
Compare against: https://hg.openjdk.java.net/jdk/jdk
Compare against version:084f13ce4eb7
Summary of changes: 1633 lines changed: 1626 ins; 0 del; 7 mod; 638 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 53690 : 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 53690 : 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
~
6 lines changed: 4 ins; 0 del; 2 mod; 52 unchg

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

@  rev 53690 : 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/DefaultStripDebugPlugin.java

@  rev 53690 : 8214796: Create a jlink plugin for stripping debug info symbols from native libraries
|  Reviewed-by: alanb, mchung, erikj
o  rev 53689 : 8218913: Rename --strip-debug jlink plugin
|  Reviewed-by: duke
~
48 lines changed: 46 ins; 0 del; 2 mod; 57 unchg

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

@  rev 53690 : 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

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/jdk/tools/jlink/IntegrationTest.java

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

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

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

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

@  rev 53690 : 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/DefaultStripDebugPluginTest.java

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

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

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

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

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

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

@  rev 53690 : 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 53690 : 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).