[richfaces-svn-commits] JBoss Rich Faces SVN: r17112 - root/commons/trunk/api/src/main/java/org/richfaces.

Author: jbalunas(a)redhat.com Date: 2010-05-18 20:33:41 -0400 (Tue, 18 May 2010) New Revision: 17112 Modified: root/commons/trunk/api/src/main/java/org/richfaces/JavaLogger.java root/commons/trunk/api/src/main/java/org/richfaces/LogFactory.java Log: Fixed two checkstyle issues from port from CDK. Modified: root/commons/trunk/api/src/main/java/org/richfaces/JavaLogger.java =================================================================== --- root/commons/trunk/api/src/main/java/org/richfaces/JavaLogger.java 2010-05-19 00:30:44 UTC (rev 17111) +++ root/commons/trunk/api/src/main/java/org/richfaces/JavaLogger.java 2010-05-19 00:33:41 UTC (rev 17112) @@ -205,7 +205,10 @@ return java.util.logging.Level.WARNING; case DEBUG: return java.util.logging.Level.FINE; + default :{ + //No level set + throw new IllegalArgumentException("Logging level must be set to obtain a logger"); + } } - return java.util.logging.Level.FINEST; } } Modified: root/commons/trunk/api/src/main/java/org/richfaces/LogFactory.java =================================================================== --- root/commons/trunk/api/src/main/java/org/richfaces/LogFactory.java 2010-05-19 00:30:44 UTC (rev 17111) +++ root/commons/trunk/api/src/main/java/org/richfaces/LogFactory.java 2010-05-19 00:33:41 UTC (rev 17112) @@ -30,7 +30,7 @@ */ public final class LogFactory { - private static final Logger defaultLogger = new JavaLogger(); + private static final Logger DEFAULT_LOGGER = new JavaLogger(); private LogFactory(){ // This class is not instantiable. @@ -42,7 +42,7 @@ * @return */ public static Logger getLogger(){ - return defaultLogger; + return DEFAULT_LOGGER; } /**