Revision: 37115
          
http://projects.blender.org/scm/viewvc.php?view=rev&root=bf-blender&revision=37115
Author:   campbellbarton
Date:     2011-06-03 04:39:18 +0000 (Fri, 03 Jun 2011)
Log Message:
-----------
disable python/string enum duplication from last commit because the array of 
duplicated strings can be freed and the pointers to the strings referenced 
still, the problem with python freeing strings that RNA references remains.

Modified Paths:
--------------
    trunk/blender/source/blender/python/intern/bpy_props.c

Modified: trunk/blender/source/blender/python/intern/bpy_props.c
===================================================================
--- trunk/blender/source/blender/python/intern/bpy_props.c      2011-06-03 
04:21:41 UTC (rev 37114)
+++ trunk/blender/source/blender/python/intern/bpy_props.c      2011-06-03 
04:39:18 UTC (rev 37115)
@@ -642,6 +642,7 @@
        Py_RETURN_NONE;
 }
 
+#if 0
 /* copies orig to buf, then sets orig to buf, returns copy length */
 static size_t strswapbufcpy(char *buf, const char **orig)
 {
@@ -652,6 +653,7 @@
        while((*dst= *src)) { dst++; src++; i++; }
        return i + 1; /* include '\0' */
 }
+#endif
 
 static EnumPropertyItem *enum_items_from_py(PyObject *seq_fast, PyObject *def, 
int *defvalue, const short is_enum_flag)
 {
@@ -760,6 +762,10 @@
                }
        }
 
+       /* disabled duplicating strings because the array can still be freed and
+        * the strings from it referenced, for now we can't support dynamically
+        * created strings from python. */
+#if 0
        /* this would all work perfectly _but_ the python strings may be freed
         * immediately after use, so we need to duplicate them, ugh.
         * annoying because it works most of the time without this. */
@@ -777,6 +783,7 @@
                items=items_dup;
        }
        /* end string duplication */
+#endif
 
        return items;
 }

_______________________________________________
Bf-blender-cvs mailing list
Bf-blender-cvs@blender.org
http://lists.blender.org/mailman/listinfo/bf-blender-cvs

Reply via email to