Author: markt Date: Thu Oct 5 12:51:53 2017 New Revision: 1811200 URL: http://svn.apache.org/viewvc?rev=1811200&view=rev Log: Fix some more deprecation warnings when building with Java 9. Use of Class.newInstance() in o.a.juli
Modified: tomcat/trunk/java/org/apache/juli/ClassLoaderLogManager.java tomcat/trunk/java/org/apache/juli/FileHandler.java tomcat/trunk/java/org/apache/juli/logging/DirectJDKLog.java Modified: tomcat/trunk/java/org/apache/juli/ClassLoaderLogManager.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/juli/ClassLoaderLogManager.java?rev=1811200&r1=1811199&r2=1811200&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/juli/ClassLoaderLogManager.java (original) +++ tomcat/trunk/java/org/apache/juli/ClassLoaderLogManager.java Thu Oct 5 12:51:53 2017 @@ -578,8 +578,8 @@ public class ClassLoaderLogManager exten } try { this.prefix.set(prefix); - Handler handler = - (Handler) classLoader.loadClass(handlerClassName).newInstance(); + Handler handler = (Handler) classLoader.loadClass( + handlerClassName).getDeclaredConstructor().newInstance(); // The specification strongly implies all configuration should be done // during the creation of the handler object. // This includes setting level, filter, formatter and encoding. Modified: tomcat/trunk/java/org/apache/juli/FileHandler.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/juli/FileHandler.java?rev=1811200&r1=1811199&r2=1811200&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/juli/FileHandler.java (original) +++ tomcat/trunk/java/org/apache/juli/FileHandler.java Thu Oct 5 12:51:53 2017 @@ -424,7 +424,7 @@ public class FileHandler extends Handler String filterName = getProperty(className + ".filter", null); if (filterName != null) { try { - setFilter((Filter) cl.loadClass(filterName).newInstance()); + setFilter((Filter) cl.loadClass(filterName).getDeclaredConstructor().newInstance()); } catch (Exception e) { // Ignore } @@ -434,7 +434,8 @@ public class FileHandler extends Handler String formatterName = getProperty(className + ".formatter", null); if (formatterName != null) { try { - setFormatter((Formatter) cl.loadClass(formatterName).newInstance()); + setFormatter((Formatter) cl.loadClass( + formatterName).getDeclaredConstructor().newInstance()); } catch (Exception e) { // Ignore and fallback to defaults setFormatter(new OneLineFormatter()); Modified: tomcat/trunk/java/org/apache/juli/logging/DirectJDKLog.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/juli/logging/DirectJDKLog.java?rev=1811200&r1=1811199&r2=1811200&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/juli/logging/DirectJDKLog.java (original) +++ tomcat/trunk/java/org/apache/juli/logging/DirectJDKLog.java Thu Oct 5 12:51:53 2017 @@ -42,11 +42,12 @@ class DirectJDKLog implements Log { // default configuration - it sucks. Let's override at least the // formatter for the console try { - Class.forName(SIMPLE_CFG).newInstance(); + Class.forName(SIMPLE_CFG).getDeclaredConstructor().newInstance(); } catch( Throwable t ) { } try { - Formatter fmt=(Formatter)Class.forName(System.getProperty(FORMATTER, SIMPLE_FMT)).newInstance(); + Formatter fmt= (Formatter) Class.forName(System.getProperty( + FORMATTER, SIMPLE_FMT)).getDeclaredConstructor().newInstance(); // it is also possible that the user modified jre/lib/logging.properties - // but that's really stupid in most cases Logger root=Logger.getLogger(""); --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org