Follow up on pull request 21. Since PyCapsule_GetPointer returns NULL on error, we should check if the list element is Py_None.

llvm/core.py

@@ -1456,6 +1456,9 @@ class Function(GlobalValue):
 class MetaData(Value):
     @staticmethod
     def get(module, values):
+        '''
+        values -- must be an iterable of Constant or None. None is treated as "null".
+        '''
         vs = unpack_values_or_none(values)
         ptr = _core.LLVMMetaDataGet(module.ptr, vs)
         return MetaData(ptr)

llvm/wrap.cpp

@@ -101,7 +101,14 @@ void **make_array_from_list(PyObject *list, int n)
     int i;
     for (i=0; i<n; i++) {
         PyObject *e = PyList_GetItem(list, i);
-        arr[i] = PyCapsule_GetPointer(e, NULL);
+        if ( e == Py_None ) { // is None object?
+            arr[i] = NULL;
+        }
+        else { // otherwise, it must be a PyCapsule
+            void * ptr = PyCapsule_GetPointer(e, NULL);
+            if ( !ptr ) return NULL; // exception is raised
+            else arr[i] = ptr;
+        }
     }
 
     return arr;