svn commit: r189543 - /jakarta/commons/proper/beanutils/trunk/src/java/org/apache/commons/beanutils/BeanUtils.java

[sk...@apache.org]

Author: skitching
Date: Wed Jun  8 00:37:10 2005
New Revision: 189543

URL: http://svn.apache.org/viewcvs?rev=189543&view=rev
Log:
Make a private variable final.
Add some comments.

Modified:
    jakarta/commons/proper/beanutils/trunk/src/java/org/apache/commons/beanutils/BeanUtils.java

Modified: jakarta/commons/proper/beanutils/trunk/src/java/org/apache/commons/beanutils/BeanUtils.java
URL: http://svn.apache.org/viewcvs/jakarta/commons/proper/beanutils/trunk/src/java/org/apache/commons/beanutils/BeanUtils.java?rev=189543&r1=189542&r2=189543&view=diff
==============================================================================
--- jakarta/commons/proper/beanutils/trunk/src/java/org/apache/commons/beanutils/BeanUtils.java (original)
+++ jakarta/commons/proper/beanutils/trunk/src/java/org/apache/commons/beanutils/BeanUtils.java Wed Jun  8 00:37:10 2005
@@ -52,10 +52,16 @@
      * Otherwise, the ClassNotFoundException thrown by ConvertUtils or
      * PropertyUtils can get masked.
      */
-    private static FastHashMap dummy = new FastHashMap();
+    private static final FastHashMap dummy = new FastHashMap();
 
     /**
      * The debugging detail level for this component.
+     * 
+     * Note that this static variable will have unexpected side-effects if
+     * this class is deployed in a shared classloader within a container.
+     * However as it is actually completely ignored by this class due to its
+     * deprecated status, it doesn't do any actual harm.
+     * 
      * @deprecated BeanUtils now uses commons-logging for all log messages.
      *             Use your favorite logging tool to configure logging for
      *             this class.



---------------------------------------------------------------------
To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: commons-dev-help@jakarta.apache.org