Code Review for jdk

Prepared by:denghui.ddh@alibaba-inc.com on Fri Sep 27 13:25:35 CST 2019
Workspace:/ssd1/home/ddh/dev/jdk8u-jfr-incubator/jdk
Compare against version:6b194072e123
Summary of changes: 377 lines changed: 373 ins; 0 del; 4 mod; 499 unchg
Patch of changes: jdk.patch
Legend: Modified file
Deleted file
New file

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/classes/jdk/jfr/internal/dcmd/DCmdStart.java

rev 13686 : 8216064: -XX:StartFlightRecording:settings= doesn't work properly
Reviewed-by: mgronlun
7 lines changed: 4 ins; 0 del; 3 mod; 257 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/classes/jdk/jfr/internal/jfc/JFC.java

rev 13686 : 8216064: -XX:StartFlightRecording:settings= doesn't work properly
Reviewed-by: mgronlun
5 lines changed: 4 ins; 0 del; 1 mod; 242 unchg

------ ------ ------ ------ --- New Patch Raw test/jdk/jfr/javaagent/EventEmitterAgent.java

rev 13687 : 8226779: [TESTBUG] Test JFR API from Java agent
Summary: Created new tests for JFR plus Java Agent
Reviewed-by: egahlin
106 lines changed: 106 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw test/jdk/jfr/javaagent/JavaAgentBuilder.java

rev 13687 : 8226779: [TESTBUG] Test JFR API from Java agent
Summary: Created new tests for JFR plus Java Agent
Reviewed-by: egahlin
87 lines changed: 87 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw test/jdk/jfr/javaagent/TestLoadedAgent.java

rev 13687 : 8226779: [TESTBUG] Test JFR API from Java agent
Summary: Created new tests for JFR plus Java Agent
Reviewed-by: egahlin
57 lines changed: 57 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw test/jdk/jfr/javaagent/TestPremainAgent.java

rev 13687 : 8226779: [TESTBUG] Test JFR API from Java agent
Summary: Created new tests for JFR plus Java Agent
Reviewed-by: egahlin
47 lines changed: 47 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw test/jdk/jfr/startupargs/TestStartNoSettings.java

rev 13686 : 8216064: -XX:StartFlightRecording:settings= doesn't work properly
Reviewed-by: mgronlun
68 lines changed: 68 ins; 0 del; 0 mod; 0 unchg

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