Code Review for jdk9-dev-platform-logging-api

Prepared by:dfuchs on Fri Oct 9 21:42:32 CEST 2015
Workspace:/Users/danielfuchs/workspaces/jdk/jdk9-dev-platform-logging-api
Summary of changes: 18577 lines changed: 17749 ins; 690 del; 138 mod; 10749 unchg
Patch of changes: jdk9-dev-platform-logging-api.patch
Legend: Modified file
Deleted file
New file

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw modules.xml

4 lines changed: 4 ins; 0 del; 0 mod; 1971 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw jdk/src/java.base/share/classes/java/lang/RuntimePermission.java

13 lines changed: 13 ins; 0 del; 0 mod; 406 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw jdk/src/java.base/share/classes/java/lang/System.java

829 lines changed: 827 ins; 2 del; 0 mod; 1274 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw jdk/src/java.base/share/classes/sun/util/logging/PlatformLogger.java

468 lines changed: 64 ins; 309 del; 95 mod; 256 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw jdk/src/java.desktop/share/classes/sun/font/FontUtilities.java

2 lines changed: 2 ins; 0 del; 0 mod; 512 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw jdk/src/java.logging/share/classes/java/util/logging/LogManager.java

34 lines changed: 32 ins; 0 del; 2 mod; 1912 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw jdk/src/java.logging/share/classes/java/util/logging/LogRecord.java

27 lines changed: 22 ins; 1 del; 4 mod; 676 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw jdk/src/java.logging/share/classes/java/util/logging/Logger.java

66 lines changed: 64 ins; 1 del; 1 mod; 2227 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw jdk/src/java.logging/share/classes/java/util/logging/LoggingPermission.java

2 lines changed: 0 ins; 0 del; 2 mod; 76 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw jdk/src/java.logging/share/classes/java/util/logging/SimpleFormatter.java

9 lines changed: 4 ins; 1 del; 4 mod; 178 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw jdk/src/java.management/share/classes/java/lang/management/DefaultPlatformMBeanProvider.java

24 lines changed: 2 ins; 0 del; 22 mod; 446 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw jdk/src/java.management/share/classes/sun/management/ManagementFactoryHelper.java

50 lines changed: 44 ins; 0 del; 6 mod; 450 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw jdk/test/java/util/logging/LoggerSubclass.java

1 line changed: 0 ins; 0 del; 1 mod; 99 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw jdk/test/sun/util/logging/PlatformLoggerTest.java

3 lines changed: 1 ins; 1 del; 1 mod; 266 unchg

------ ------ ------ ------ --- New Patch Raw jdk/src/java.base/share/classes/sun/util/logger/AbstractLoggerWrapper.java

380 lines changed: 380 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw jdk/src/java.base/share/classes/sun/util/logger/BootstrapLogger.java

1028 lines changed: 1028 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw jdk/src/java.base/share/classes/sun/util/logger/JdkLoggerProvider.java

165 lines changed: 165 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw jdk/src/java.base/share/classes/sun/util/logger/LazyLoggers.java

483 lines changed: 483 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw jdk/src/java.base/share/classes/sun/util/logger/LocalizedLoggerWrapper.java

156 lines changed: 156 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw jdk/src/java.base/share/classes/sun/util/logger/LoggerWrapper.java

67 lines changed: 67 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw jdk/src/java.base/share/classes/sun/util/logger/SimpleConsoleLogger.java

484 lines changed: 484 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw jdk/src/java.base/share/classes/sun/util/logger/package-info.java

89 lines changed: 89 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw jdk/src/java.base/share/classes/sun/util/logging/ConfigurableLoggerBridge.java

70 lines changed: 70 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw jdk/src/java.base/share/classes/sun/util/logging/PlatformLoggerBridge.java

100 lines changed: 100 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw jdk/src/java.logging/share/classes/META-INF/services/sun.util.logger.JdkLoggerProvider

1 line changed: 1 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw jdk/src/java.logging/share/classes/sun/util/logging/internal/JdkLoggingProvider.java

425 lines changed: 425 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw jdk/src/java.logging/share/classes/sun/util/logging/internal/package-info.java

55 lines changed: 55 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw jdk/test/java/lang/System/Logger/custom/AccessSystemLogger.java

82 lines changed: 82 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw jdk/test/java/lang/System/Logger/custom/CustomLoggerTest.java

726 lines changed: 726 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw jdk/test/java/lang/System/Logger/custom/META-INF/services/java.lang.System$LoggerFinder

1 line changed: 1 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw jdk/test/java/lang/System/Logger/default/AccessSystemLogger.java

82 lines changed: 82 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw jdk/test/java/lang/System/Logger/default/DefaultLoggerTest.java

726 lines changed: 726 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw jdk/test/java/lang/System/Logger/interface/LoggerInterfaceTest.java

489 lines changed: 489 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw jdk/test/java/lang/System/Logger/Level/LoggerLevelTest.java

87 lines changed: 87 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw jdk/test/java/lang/System/LoggerFinder/internal/BaseLoggerBridgeTest/BaseLoggerBridgeTest.java

1055 lines changed: 1055 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw jdk/test/java/lang/System/LoggerFinder/internal/BaseLoggerBridgeTest/CustomSystemClassLoader.java

101 lines changed: 101 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw jdk/test/java/lang/System/LoggerFinder/internal/BaseLoggerBridgeTest/META-INF/services/java.lang.System$LoggerFinder

1 line changed: 1 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw jdk/test/java/lang/System/LoggerFinder/internal/BasePlatformLoggerTest/BasePlatformLoggerTest.java

730 lines changed: 730 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw jdk/test/java/lang/System/LoggerFinder/internal/BasePlatformLoggerTest/CustomSystemClassLoader.java

101 lines changed: 101 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw jdk/test/java/lang/System/LoggerFinder/internal/BasePlatformLoggerTest/META-INF/services/java.lang.System$LoggerFinder

1 line changed: 1 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw jdk/test/java/lang/System/LoggerFinder/internal/BootstrapLogger/BootstrapLoggerTest.java

428 lines changed: 428 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw jdk/test/java/lang/System/LoggerFinder/internal/LoggerBridgeTest/CustomSystemClassLoader.java

141 lines changed: 141 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw jdk/test/java/lang/System/LoggerFinder/internal/LoggerBridgeTest/LoggerBridgeTest.java

1086 lines changed: 1086 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw jdk/test/java/lang/System/LoggerFinder/internal/LoggerBridgeTest/META-INF/services/java.lang.System$LoggerFinder

1 line changed: 1 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw jdk/test/java/lang/System/LoggerFinder/internal/PlatformLoggerBridgeTest/CustomSystemClassLoader.java

101 lines changed: 101 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw jdk/test/java/lang/System/LoggerFinder/internal/PlatformLoggerBridgeTest/META-INF/services/java.lang.System$LoggerFinder

1 line changed: 1 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw jdk/test/java/lang/System/LoggerFinder/internal/PlatformLoggerBridgeTest/PlatformLoggerBridgeTest.java

876 lines changed: 876 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw jdk/test/java/lang/System/LoggerFinder/internal/api/LoggerFinderAPITest.java

498 lines changed: 498 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw jdk/test/java/lang/System/LoggerFinder/internal/backend/LoggerFinderBackendTest.java

2313 lines changed: 2313 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw jdk/test/java/lang/System/LoggerFinder/internal/backend/META-INF/services/java.lang.System$LoggerFinder

2 lines changed: 2 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw jdk/test/java/lang/System/LoggerFinder/internal/backend/SystemClassLoader.java

83 lines changed: 83 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw jdk/test/java/lang/System/LoggerFinder/jdk/DefaultLoggerBridgeTest/DefaultLoggerBridgeTest.java

844 lines changed: 844 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw jdk/test/java/lang/System/LoggerFinder/jdk/DefaultPlatformLoggerTest/DefaultPlatformLoggerTest.java

540 lines changed: 540 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw jdk/test/java/lang/System/LoggerFinder/public/BaseLoggerFinderTest/AccessSystemLogger.java

82 lines changed: 82 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw jdk/test/java/lang/System/LoggerFinder/public/BaseLoggerFinderTest/BaseLoggerFinder.java

46 lines changed: 46 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw jdk/test/java/lang/System/LoggerFinder/public/BaseLoggerFinderTest/BaseLoggerFinderTest.java

692 lines changed: 692 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw jdk/test/java/lang/System/LoggerFinder/public/BaseLoggerFinderTest/CustomSystemClassLoader.java

101 lines changed: 101 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw jdk/test/java/lang/System/LoggerFinder/public/BaseLoggerFinderTest/META-INF/services/java.lang.System$LoggerFinder

1 line changed: 1 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw jdk/test/java/lang/System/LoggerFinder/public/BaseLoggerFinderTest/TestLoggerFinder.java

181 lines changed: 181 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw jdk/test/java/lang/System/LoggerFinder/public/DefaultLoggerFinderTest/AccessSystemLogger.java

82 lines changed: 82 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw jdk/test/java/lang/System/LoggerFinder/public/DefaultLoggerFinderTest/DefaultLoggerFinderTest.java

886 lines changed: 886 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ Old --- Patch --- jdk/src/java.base/share/classes/sun/util/logging/LoggingProxy.java

68 lines changed: 0 ins; 68 del; 0 mod; 0 unchg

------ ------ ------ ------ Old --- Patch --- jdk/src/java.base/share/classes/sun/util/logging/LoggingSupport.java

190 lines changed: 0 ins; 190 del; 0 mod; 0 unchg

------ ------ ------ ------ Old --- Patch --- jdk/src/java.logging/share/classes/java/util/logging/LoggingProxyImpl.java

117 lines changed: 0 ins; 117 del; 0 mod; 0 unchg

This code review page was prepared using /Users/danielfuchs/bin/webrev (vers 25.12-hg+openjdk.java.net).