Skip to content

Commit

Permalink
Merge pull request #18462 from pshipton/logger11
Browse files Browse the repository at this point in the history
jdk11 - Don't cache instances of TemporaryLoggerFinder
  • Loading branch information
keithc-ca authored Nov 16, 2023
2 parents e95661e + f924dc8 commit 5899f3c
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions jcl/src/java.base/share/classes/java/lang/System.java
Original file line number Diff line number Diff line change
Expand Up @@ -1802,13 +1802,13 @@ public static LoggerFinder getLoggerFinder() {
(PrivilegedAction<LoggerFinder>) () -> jdk.internal.logger.LoggerFinderLoader.getLoggerFinder(),
AccessController.getContext(),
com.ibm.oti.util.RuntimePermissions.permissionLoggerFinder);
/*[IF JAVA_SPEC_VERSION >= 17]*/
/*[IF JAVA_SPEC_VERSION >= 11]*/
/*[IF JAVA_SPEC_VERSION != 21] Temporary until jdk21 picks up the OpenJDK change */
if (localFinder instanceof jdk.internal.logger.LoggerFinderLoader.TemporaryLoggerFinder) {
return localFinder;
}
/*[ENDIF] JAVA_SPEC_VERSION != 21 */
/*[ENDIF] JAVA_SPEC_VERSION >= 17 */
/*[ENDIF] JAVA_SPEC_VERSION >= 11 */
loggerFinder = localFinder;
}
return localFinder;
Expand Down

0 comments on commit 5899f3c

Please sign in to comment.