Revision: 459
          http://svn.sourceforge.net/stripes/?rev=459&view=rev
Author:   tfenne
Date:     2006-10-28 08:30:22 -0700 (Sat, 28 Oct 2006)

Log Message:
-----------
Sweep to ensure that all Log instances are declared as private static final

Modified Paths:
--------------
    trunk/stripes/src/net/sourceforge/stripes/action/ActionBeanContext.java
    trunk/stripes/src/net/sourceforge/stripes/ajax/JavaScriptBuilder.java
    
trunk/stripes/src/net/sourceforge/stripes/controller/AnnotatedClassActionResolver.java
    trunk/stripes/src/net/sourceforge/stripes/controller/DispatcherServlet.java
    trunk/stripes/src/net/sourceforge/stripes/controller/StripesFilter.java
    
trunk/stripes/src/net/sourceforge/stripes/integration/spring/SpringHelper.java
    
trunk/stripes/src/net/sourceforge/stripes/localization/DefaultLocalePicker.java
    
trunk/stripes/src/net/sourceforge/stripes/localization/LocalizationUtility.java
    
trunk/stripes/src/net/sourceforge/stripes/tag/BeanFirstPopulationStrategy.java
    trunk/stripes/src/net/sourceforge/stripes/tag/DefaultPopulationStrategy.java
    trunk/stripes/src/net/sourceforge/stripes/tag/ErrorsTag.java
    trunk/stripes/src/net/sourceforge/stripes/tag/FormTag.java
    trunk/stripes/src/net/sourceforge/stripes/tag/HtmlTagSupport.java
    trunk/stripes/src/net/sourceforge/stripes/tag/IndividualErrorTag.java
    
trunk/stripes/src/net/sourceforge/stripes/tag/layout/LayoutDefinitionTag.java
    trunk/stripes/src/net/sourceforge/stripes/validation/SimpleError.java

Modified: 
trunk/stripes/src/net/sourceforge/stripes/action/ActionBeanContext.java
===================================================================
--- trunk/stripes/src/net/sourceforge/stripes/action/ActionBeanContext.java     
2006-10-28 14:56:58 UTC (rev 458)
+++ trunk/stripes/src/net/sourceforge/stripes/action/ActionBeanContext.java     
2006-10-28 15:30:22 UTC (rev 459)
@@ -14,17 +14,16 @@
  */
 package net.sourceforge.stripes.action;
 
-import net.sourceforge.stripes.validation.ValidationErrors;
-import net.sourceforge.stripes.controller.StripesConstants;
 import net.sourceforge.stripes.controller.FlashScope;
-import net.sourceforge.stripes.util.Log;
+import net.sourceforge.stripes.controller.StripesConstants;
+import net.sourceforge.stripes.validation.ValidationErrors;
 
+import javax.servlet.ServletContext;
 import javax.servlet.http.HttpServletRequest;
 import javax.servlet.http.HttpServletResponse;
-import javax.servlet.ServletContext;
-import java.util.Locale;
-import java.util.List;
 import java.util.ArrayList;
+import java.util.List;
+import java.util.Locale;
 
 /**
  * <p>Encapsulates information about the current request.  Also provides 
access to the underlying
@@ -39,8 +38,6 @@
  * @author Tim Fennell
  */
 public class ActionBeanContext {
-    private static Log log = Log.getInstance(ActionBeanContext.class);
-
     private HttpServletRequest request;
     private HttpServletResponse response;
     private ServletContext servletContext;

Modified: trunk/stripes/src/net/sourceforge/stripes/ajax/JavaScriptBuilder.java
===================================================================
--- trunk/stripes/src/net/sourceforge/stripes/ajax/JavaScriptBuilder.java       
2006-10-28 14:56:58 UTC (rev 458)
+++ trunk/stripes/src/net/sourceforge/stripes/ajax/JavaScriptBuilder.java       
2006-10-28 15:30:22 UTC (rev 459)
@@ -48,7 +48,7 @@
  */
 public class JavaScriptBuilder {
     /** Log instance used to log messages. */
-    static Log log = Log.getInstance(JavaScriptBuilder.class);
+    private static Log log = Log.getInstance(JavaScriptBuilder.class);
 
     /** Holds the set of classes representing the primitive types in Java. */
     static Set<Class<?>> simpleTypes = new HashSet<Class<?>>();

Modified: 
trunk/stripes/src/net/sourceforge/stripes/controller/AnnotatedClassActionResolver.java
===================================================================
--- 
trunk/stripes/src/net/sourceforge/stripes/controller/AnnotatedClassActionResolver.java
      2006-10-28 14:56:58 UTC (rev 458)
+++ 
trunk/stripes/src/net/sourceforge/stripes/controller/AnnotatedClassActionResolver.java
      2006-10-28 15:30:22 UTC (rev 459)
@@ -71,7 +71,7 @@
     private static final String DEFAULT_HANDLER_KEY = "__default_handler";
 
     /** Log instance for use within in this class. */
-    private Log log = Log.getInstance(AnnotatedClassActionResolver.class);
+    private static final Log log = 
Log.getInstance(AnnotatedClassActionResolver.class);
 
     /** Handle to the configuration. */
     private Configuration configuration;

Modified: 
trunk/stripes/src/net/sourceforge/stripes/controller/DispatcherServlet.java
===================================================================
--- trunk/stripes/src/net/sourceforge/stripes/controller/DispatcherServlet.java 
2006-10-28 14:56:58 UTC (rev 458)
+++ trunk/stripes/src/net/sourceforge/stripes/controller/DispatcherServlet.java 
2006-10-28 15:30:22 UTC (rev 459)
@@ -53,7 +53,7 @@
     private Boolean alwaysInvokeValidate;
 
     /** Log used throughout the class. */
-    private static Log log = Log.getInstance(DispatcherServlet.class);
+    private static final Log log = Log.getInstance(DispatcherServlet.class);
 
     /** Implemented as a simple call to doPost(request, response). */
     protected void doGet(HttpServletRequest request, HttpServletResponse 
response)

Modified: 
trunk/stripes/src/net/sourceforge/stripes/controller/StripesFilter.java
===================================================================
--- trunk/stripes/src/net/sourceforge/stripes/controller/StripesFilter.java     
2006-10-28 14:56:58 UTC (rev 458)
+++ trunk/stripes/src/net/sourceforge/stripes/controller/StripesFilter.java     
2006-10-28 15:30:22 UTC (rev 459)
@@ -50,7 +50,7 @@
     public static final String CONFIG_CLASS = "Configuration.Class";
 
     /** Log used throughout the class. */
-    private static Log log = Log.getInstance(StripesFilter.class);
+    private static final Log log = Log.getInstance(StripesFilter.class);
 
     /** The configuration instance for Stripes. */
     private Configuration configuration;

Modified: 
trunk/stripes/src/net/sourceforge/stripes/integration/spring/SpringHelper.java
===================================================================
--- 
trunk/stripes/src/net/sourceforge/stripes/integration/spring/SpringHelper.java  
    2006-10-28 14:56:58 UTC (rev 458)
+++ 
trunk/stripes/src/net/sourceforge/stripes/integration/spring/SpringHelper.java  
    2006-10-28 15:30:22 UTC (rev 459)
@@ -58,7 +58,7 @@
  * @author Dan Hayes, Tim Fennell
  */
 public class SpringHelper {
-    private static Log log = Log.getInstance(SpringHelper.class);
+    private static final Log log = Log.getInstance(SpringHelper.class);
 
     /** Lazily filled in map of Class to methods annotated with SpringBean. */
     private static Map<Class<?>, Collection<Method>> methodMap =

Modified: 
trunk/stripes/src/net/sourceforge/stripes/localization/DefaultLocalePicker.java
===================================================================
--- 
trunk/stripes/src/net/sourceforge/stripes/localization/DefaultLocalePicker.java 
    2006-10-28 14:56:58 UTC (rev 458)
+++ 
trunk/stripes/src/net/sourceforge/stripes/localization/DefaultLocalePicker.java 
    2006-10-28 15:30:22 UTC (rev 459)
@@ -51,7 +51,7 @@
     public static final String LOCALE_LIST = "LocalePicker.Locales";
 
     /** Log instance for use within the class. */
-    private final Log log = Log.getInstance(DefaultLocalePicker.class);
+    private static final Log log = Log.getInstance(DefaultLocalePicker.class);
 
     /** Stores a reference to the configuration passed in at initialization. */
     protected Configuration configuration;

Modified: 
trunk/stripes/src/net/sourceforge/stripes/localization/LocalizationUtility.java
===================================================================
--- 
trunk/stripes/src/net/sourceforge/stripes/localization/LocalizationUtility.java 
    2006-10-28 14:56:58 UTC (rev 458)
+++ 
trunk/stripes/src/net/sourceforge/stripes/localization/LocalizationUtility.java 
    2006-10-28 15:30:22 UTC (rev 459)
@@ -31,7 +31,7 @@
  * @since Stripes 1.1
  */
 public class LocalizationUtility {
-    private static Log log = Log.getInstance(LocalizationUtility.class);
+    private static final Log log = Log.getInstance(LocalizationUtility.class);
 
     /**
      * <p>Fetches the localized name for a form field if one exists in the 
form field resource bundle.

Modified: 
trunk/stripes/src/net/sourceforge/stripes/tag/BeanFirstPopulationStrategy.java
===================================================================
--- 
trunk/stripes/src/net/sourceforge/stripes/tag/BeanFirstPopulationStrategy.java  
    2006-10-28 14:56:58 UTC (rev 458)
+++ 
trunk/stripes/src/net/sourceforge/stripes/tag/BeanFirstPopulationStrategy.java  
    2006-10-28 15:30:22 UTC (rev 459)
@@ -35,7 +35,7 @@
  * @since Stripes 1.4
  */
 public class BeanFirstPopulationStrategy extends DefaultPopulationStrategy {
-    private static Log log = 
Log.getInstance(BeanFirstPopulationStrategy.class);
+    private static final Log log = 
Log.getInstance(BeanFirstPopulationStrategy.class);
 
     /**
      * Implementation of the interface method that will follow the search 
described in the class

Modified: 
trunk/stripes/src/net/sourceforge/stripes/tag/DefaultPopulationStrategy.java
===================================================================
--- 
trunk/stripes/src/net/sourceforge/stripes/tag/DefaultPopulationStrategy.java    
    2006-10-28 14:56:58 UTC (rev 458)
+++ 
trunk/stripes/src/net/sourceforge/stripes/tag/DefaultPopulationStrategy.java    
    2006-10-28 15:30:22 UTC (rev 459)
@@ -38,7 +38,7 @@
     private Configuration config;
 
     /** Log used to log any errors that occur. */
-    private static Log log = Log.getInstance(DefaultPopulationStrategy.class);
+    private static final Log log = 
Log.getInstance(DefaultPopulationStrategy.class);
 
     /** Called by the Configuration to configure the component. */
     public void init(Configuration configuration) throws Exception {

Modified: trunk/stripes/src/net/sourceforge/stripes/tag/ErrorsTag.java
===================================================================
--- trunk/stripes/src/net/sourceforge/stripes/tag/ErrorsTag.java        
2006-10-28 14:56:58 UTC (rev 458)
+++ trunk/stripes/src/net/sourceforge/stripes/tag/ErrorsTag.java        
2006-10-28 15:30:22 UTC (rev 459)
@@ -85,7 +85,7 @@
  */
 public class ErrorsTag extends HtmlTagSupport implements BodyTag {
 
-    private final Log log = Log.getInstance(ErrorsTag.class);
+    private static final Log log = Log.getInstance(ErrorsTag.class);
 
     /** The header that will be emitted if no header is defined in the 
resource bundle. */
     public static final String DEFAULT_HEADER =

Modified: trunk/stripes/src/net/sourceforge/stripes/tag/FormTag.java
===================================================================
--- trunk/stripes/src/net/sourceforge/stripes/tag/FormTag.java  2006-10-28 
14:56:58 UTC (rev 458)
+++ trunk/stripes/src/net/sourceforge/stripes/tag/FormTag.java  2006-10-28 
15:30:22 UTC (rev 459)
@@ -47,7 +47,7 @@
  */
 public class FormTag extends HtmlTagSupport implements BodyTag, 
TryCatchFinally {
     /** Log used to log error and debugging information for this class. */
-    private static Log log = Log.getInstance(FormTag.class);
+    private static final Log log = Log.getInstance(FormTag.class);
 
     /** Stores the field name (or magic values ''/'first') to set focus on. */
     private String focus;

Modified: trunk/stripes/src/net/sourceforge/stripes/tag/HtmlTagSupport.java
===================================================================
--- trunk/stripes/src/net/sourceforge/stripes/tag/HtmlTagSupport.java   
2006-10-28 14:56:58 UTC (rev 458)
+++ trunk/stripes/src/net/sourceforge/stripes/tag/HtmlTagSupport.java   
2006-10-28 15:30:22 UTC (rev 459)
@@ -37,7 +37,7 @@
  */
 public abstract class HtmlTagSupport extends StripesTagSupport implements 
DynamicAttributes {
     /** Log implementation used to log errors during tag writing. */
-    private final Log log = Log.getInstance(HtmlTagSupport.class);
+    private static final Log log = Log.getInstance(HtmlTagSupport.class);
 
     /** Map containing all attributes of the tag. */
     private final Map<String,String> attributes = new HashMap<String,String>();

Modified: trunk/stripes/src/net/sourceforge/stripes/tag/IndividualErrorTag.java
===================================================================
--- trunk/stripes/src/net/sourceforge/stripes/tag/IndividualErrorTag.java       
2006-10-28 14:56:58 UTC (rev 458)
+++ trunk/stripes/src/net/sourceforge/stripes/tag/IndividualErrorTag.java       
2006-10-28 15:30:22 UTC (rev 459)
@@ -33,7 +33,7 @@
  */
 public class IndividualErrorTag extends HtmlTagSupport implements Tag {
 
-    private final Log log = Log.getInstance(ErrorsTag.class);
+    private static final Log log = Log.getInstance(ErrorsTag.class);
 
     /**
      * Does nothing

Modified: 
trunk/stripes/src/net/sourceforge/stripes/tag/layout/LayoutDefinitionTag.java
===================================================================
--- 
trunk/stripes/src/net/sourceforge/stripes/tag/layout/LayoutDefinitionTag.java   
    2006-10-28 14:56:58 UTC (rev 458)
+++ 
trunk/stripes/src/net/sourceforge/stripes/tag/layout/LayoutDefinitionTag.java   
    2006-10-28 15:30:22 UTC (rev 459)
@@ -31,7 +31,7 @@
  * @since Stripes 1.1
  */
 public class LayoutDefinitionTag extends StripesTagSupport {
-    private static Log log = Log.getInstance(LayoutDefinitionTag.class);
+    private static final Log log = Log.getInstance(LayoutDefinitionTag.class);
 
     /**
      * Prefix used to construct the request attribute name used to pass 
context from the

Modified: trunk/stripes/src/net/sourceforge/stripes/validation/SimpleError.java
===================================================================
--- trunk/stripes/src/net/sourceforge/stripes/validation/SimpleError.java       
2006-10-28 14:56:58 UTC (rev 458)
+++ trunk/stripes/src/net/sourceforge/stripes/validation/SimpleError.java       
2006-10-28 15:30:22 UTC (rev 459)
@@ -50,7 +50,7 @@
  * @see java.text.MessageFormat
  */
 public class SimpleError extends SimpleMessage implements ValidationError {
-    private static Log log = Log.getInstance(SimpleError.class);
+    private static final Log log = Log.getInstance(SimpleError.class);
 
     private String fieldNameKey;
     private String actionPath;


This was sent by the SourceForge.net collaborative development platform, the 
world's largest Open Source development site.

-------------------------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
_______________________________________________
Stripes-development mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/stripes-development

Reply via email to