Log message for revision 108729:
  Changed the class declaration inside the record_cls_factory to use the 
builtin 'type' class constructor to  avoid any chance of outer scope references 
leaking in closure-style (this is probably over paranoid but it can't hurt).

Changed:
  U   
Zope/branches/zeomega-2.11-memory-fixes/lib/python/Shared/DC/ZRDB/Results.py

-=-
Modified: 
Zope/branches/zeomega-2.11-memory-fixes/lib/python/Shared/DC/ZRDB/Results.py
===================================================================
--- 
Zope/branches/zeomega-2.11-memory-fixes/lib/python/Shared/DC/ZRDB/Results.py    
    2010-02-02 23:08:18 UTC (rev 108728)
+++ 
Zope/branches/zeomega-2.11-memory-fixes/lib/python/Shared/DC/ZRDB/Results.py    
    2010-02-03 00:47:07 UTC (rev 108729)
@@ -10,6 +10,8 @@
 # FOR A PARTICULAR PURPOSE
 #
 ##############################################################################
+
+
 import ExtensionClass
 from Acquisition import Implicit
 from Record import Record
@@ -25,10 +27,7 @@
     """Return a custom 'record' class inheriting from Record, Implicit,
     brains, and zbrains).
     """
-    # alternate implementation
-    # r = type('r', (Record, Implicit, brains, zbrains), {})
-    class r(Record, Implicit, brains, zbrains):
-        'Result record class'
+    r = type('r', (Record, Implicit, brains, zbrains), {})
 
     # The Record class needs a __record_schema__ ...why?
     r.__record_schema__=schema

_______________________________________________
Zope-Checkins maillist  -  Zope-Checkins@zope.org
https://mail.zope.org/mailman/listinfo/zope-checkins

Reply via email to