Code Review for rt

Prepared by:kcr on Fri Dec 2 13:14:10 PST 2016
Workspace:/localhome/kcr/javafx/9-kcr/jfx/rt
Compare against: ssh://hg.openjdk.java.net/openjfx/9-dev/rt
Compare against version:10172
Summary of changes: 472 lines changed: 11 ins; 434 del; 27 mod; 4823 unchg
Changeset: rt.changeset
Legend: Modified file
Deleted file
New file

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw build.gradle

rev 10173 : 8170485: Switch to building JavaFX with new module-info syntax
Reviewed-by:
31 lines changed: 0 ins; 31 del; 0 mod; 4281 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw build.properties

rev 10173 : 8170485: Switch to building JavaFX with new module-info syntax
Reviewed-by:
1 line changed: 1 ins; 0 del; 0 mod; 70 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw modules/javafx.base/src/test/addExports

rev 10173 : 8170485: Switch to building JavaFX with new module-info syntax
Reviewed-by:
3 lines changed: 3 ins; 0 del; 0 mod; 9 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw modules/javafx.controls/src/main/java/module-info.java

rev 10173 : 8170485: Switch to building JavaFX with new module-info syntax
Reviewed-by:
2 lines changed: 0 ins; 0 del; 2 mod; 42 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw modules/javafx.controls/src/test/addExports

rev 10173 : 8170485: Switch to building JavaFX with new module-info syntax
Reviewed-by:
3 lines changed: 1 ins; 0 del; 2 mod; 30 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw modules/javafx.fxml/src/main/java/module-info.java

rev 10173 : 8170485: Switch to building JavaFX with new module-info syntax
Reviewed-by:
1 line changed: 0 ins; 0 del; 1 mod; 35 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw modules/javafx.graphics/src/main/java/module-info.java

rev 10173 : 8170485: Switch to building JavaFX with new module-info syntax
Reviewed-by:
1 line changed: 0 ins; 0 del; 1 mod; 139 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw modules/javafx.graphics/src/test/addExports

rev 10173 : 8170485: Switch to building JavaFX with new module-info syntax
Reviewed-by:
2 lines changed: 2 ins; 0 del; 0 mod; 58 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw modules/javafx.media/src/main/java/module-info.java

rev 10173 : 8170485: Switch to building JavaFX with new module-info syntax
Reviewed-by:
2 lines changed: 0 ins; 0 del; 2 mod; 42 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw modules/javafx.swing/src/main/java/module-info.java

rev 10173 : 8170485: Switch to building JavaFX with new module-info syntax
Reviewed-by:
2 lines changed: 0 ins; 0 del; 2 mod; 35 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw modules/javafx.web/src/main/java/module-info.java

rev 10173 : 8170485: Switch to building JavaFX with new module-info syntax
Reviewed-by:
4 lines changed: 0 ins; 0 del; 4 mod; 38 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw modules/jdk.packager/src/main/java/module-info.java

rev 10173 : 8170485: Switch to building JavaFX with new module-info syntax
Reviewed-by:
16 lines changed: 1 ins; 2 del; 13 mod; 44 unchg

------ ------ ------ ------ --- New ----- Raw README-java-options

rev 10173 : 8170485: Switch to building JavaFX with new module-info syntax
Reviewed-by:
3 lines changed: 3 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ Old --- ----- --- modules/javafx.controls/src/main/jake/module-info.java

rev 10173 : 8170485: Switch to building JavaFX with new module-info syntax
Reviewed-by:
44 lines changed: 0 ins; 44 del; 0 mod; 0 unchg

------ ------ ------ ------ Old --- ----- --- modules/javafx.fxml/src/main/jake/module-info.java

rev 10173 : 8170485: Switch to building JavaFX with new module-info syntax
Reviewed-by:
36 lines changed: 0 ins; 36 del; 0 mod; 0 unchg

------ ------ ------ ------ Old --- ----- --- modules/javafx.graphics/src/main/jake/module-info.java

rev 10173 : 8170485: Switch to building JavaFX with new module-info syntax
Reviewed-by:
140 lines changed: 0 ins; 140 del; 0 mod; 0 unchg

------ ------ ------ ------ Old --- ----- --- modules/javafx.media/src/main/jake/module-info.java

rev 10173 : 8170485: Switch to building JavaFX with new module-info syntax
Reviewed-by:
44 lines changed: 0 ins; 44 del; 0 mod; 0 unchg

------ ------ ------ ------ Old --- ----- --- modules/javafx.swing/src/main/jake/module-info.java

rev 10173 : 8170485: Switch to building JavaFX with new module-info syntax
Reviewed-by:
37 lines changed: 0 ins; 37 del; 0 mod; 0 unchg

------ ------ ------ ------ Old --- ----- --- modules/javafx.web/src/main/jake/module-info.java

rev 10173 : 8170485: Switch to building JavaFX with new module-info syntax
Reviewed-by:
42 lines changed: 0 ins; 42 del; 0 mod; 0 unchg

------ ------ ------ ------ Old --- ----- --- modules/jdk.packager/src/main/jake/module-info.java

rev 10173 : 8170485: Switch to building JavaFX with new module-info syntax
Reviewed-by:
58 lines changed: 0 ins; 58 del; 0 mod; 0 unchg

This code review page was prepared using /localhome/kcr/bin/webrev (vers 25.16-hg+openjdk.java.net).