Author: michiel
Date: 2010-04-13 12:12:34 +0200 (Tue, 13 Apr 2010)
New Revision: 41870

Modified:
   
mmbase/trunk/core/src/main/java/org/mmbase/bridge/implementation/BasicNodeManager.java
Log:
details

Modified: 
mmbase/trunk/core/src/main/java/org/mmbase/bridge/implementation/BasicNodeManager.java
===================================================================
--- 
mmbase/trunk/core/src/main/java/org/mmbase/bridge/implementation/BasicNodeManager.java
      2010-04-13 10:12:28 UTC (rev 41869)
+++ 
mmbase/trunk/core/src/main/java/org/mmbase/bridge/implementation/BasicNodeManager.java
      2010-04-13 10:12:34 UTC (rev 41870)
@@ -53,7 +53,7 @@
     protected MMObjectBuilder builder;
 
     // field types
-    protected Map<String, Field> fieldTypes = new HashMap<String, Field>();
+    protected final Map<String, Field> fieldTypes = new HashMap<String, 
Field>();
 
     /**
      * Instantiates a new NodeManager (for insert) based on a newly created 
node which either represents or references a builder.
@@ -639,7 +639,7 @@
     protected void finalize() {
         // http://www.fasterj.com/articles/finalizer1.shtml
         // Having a non-empty finalizer can be quite expensive.
-        // BasicNodeManagers can exist very many (probably as virtual node 
manager).
+        // BasicNodeManagers can exist very many (every cloud object needs a 
bunch),
         //
         // Probably it is not actually correct to skip the finalize() of super,
         // but I think only if the node is being edited, and commit or cancel 
is not called.

_______________________________________________
Cvs mailing list
[email protected]
http://lists.mmbase.org/mailman/listinfo/cvs

Reply via email to