Don't generate conversion code for wxLists if not requested to.

Author: amauryfa

cffi/wrapper_lib/__init__.py

@@ -2,6 +2,7 @@
     CppWrapper, WrapperType, VirtualMethod, VirtualMethodStub,
     VirtualDispatcher, register_cpp_classname, obj_from_ptr, get_ptr,
     forget_ptr, take_ownership, give_ownership, keep_reference, MappedBase,
+    is_alive,
     instancecheck, convert_to_type, init_wrapper, hassubclass, CastData)
 from multimethod import (
     MMTypeCheckMeta, MMTypeError, MMInternalError, check_arg_type,

etgtools/tweaker_tools.py

@@ -940,37 +940,6 @@ class {ListClass}_iterator {{
                               "sequence.index(x): x not in sequence");
         }}
         return idx;""".format(**locals())))
-    c.addItem(extractors.CppMethodDef_cffi(
-        '_new', isStatic=True,
-        pyArgs=extractors.ArgsString('(WL_Object elements)'),
-        pyBody="""\
-        wrapper_lib.check_args_types(
-            ({0}._pyobject_mapping_, elements, "elements"))
-
-        keepalive = []
-        ptrs = []
-        for item in elements:
-            if not isinstance(item, {1}):
-                item = {1}._pyobject_mapping_.convert(item)
-                keepalive.append(item)
-            ptrs.append(wrapper_lib.get_ptr(item))
-
-        ptr = call(len(ptrs), ffi.new('void*[]', ptrs))
-        return wrapper_lib.obj_from_ptr(ptr, {0}, True)
-        """.format(ListClass_pyName, ItemClass_pyName),
-        cReturnType='void *',
-        cArgsString='(size_t count, void **elements)',
-        # Don't send data to the constructor! This calls virtual methods,
-        # which are not yet defined in the base C++ class... wx is broken.
-        cBody='''
-        {ListClass} *list = new {ListClass}; 
-        list->DeleteContents(true); // tell the list to take ownership of the items
-        for (Py_ssize_t i = 0; i < count; i++) {{
-            list->Append(new {ItemClass}(*({ItemClass}*)elements[i]));
-        }}
-        return list;
-        '''.format(**locals()),
-    ))
     # TODO:  add support for index(value, [start, [stop]])
     c.addPyMethod(
         '__repr__', '(self)',
@@ -1034,6 +1003,37 @@ class {ListClass}_iterator {{
         *sipCppPtr = list;
         return SIP_TEMPORARY;
         '''.format(**locals())
+        c.addItem(extractors.CppMethodDef_cffi(
+            '_new', isStatic=True,
+            pyArgs=extractors.ArgsString('(WL_Object elements)'),
+            pyBody="""\
+            wrapper_lib.check_args_types(
+                ({0}._pyobject_mapping_, elements, "elements"))
+
+            keepalive = []
+            ptrs = []
+            for item in elements:
+                if not isinstance(item, {1}):
+                    item = {1}._pyobject_mapping_.convert(item)
+                    keepalive.append(item)
+                ptrs.append(wrapper_lib.get_ptr(item))
+
+            ptr = call(len(ptrs), ffi.new('void*[]', ptrs))
+            return wrapper_lib.obj_from_ptr(ptr, {0}, True)
+            """.format(ListClass_pyName, ItemClass_pyName),
+            cReturnType='void *',
+            cArgsString='(size_t count, void **elements)',
+            # Don't send data to the constructor! This calls virtual methods,
+            # which are not yet defined in the base C++ class... wx is broken.
+            cBody='''
+            {ListClass} *list = new {ListClass}; 
+            list->DeleteContents(true); // tell the list to take ownership of the items
+            for (Py_ssize_t i = 0; i < count; i++) {{
+                list->Append(new {ItemClass}(*({ItemClass}*)elements[i]));
+            }}
+            return list;
+            '''.format(**locals()),
+        ))
 
     c.instanceCheck_cffi = """\
     if (not isinstance(py_obj, collections.Sequence) or