Use XINCREF in copy constructor and subview constructor, since arr may be NULL

mdboom · mdboom · commit f94ffd798c3a · 2014-11-17T16:37:27.000-05:00
diff --git a/src/numpy_cpp.h b/src/numpy_cpp.h
@@ -370,7 +370,7 @@ class array_view : public detail::array_view_accessors<array_view, T, ND>
     array_view(const array_view &other, bool contiguous = false) : m_arr(NULL), m_data(NULL)
     {
         m_arr = other.m_arr;
-        Py_INCREF(m_arr);
+        Py_XINCREF(m_arr);
         m_data = other.m_data;
         m_shape = other.m_shape;
         m_strides = other.m_strides;
@@ -379,7 +379,7 @@ class array_view : public detail::array_view_accessors<array_view, T, ND>
     array_view(PyArrayObject *arr, char *data, npy_intp *shape, npy_intp *strides)
     {
         m_arr = arr;
-        Py_INCREF(arr);
+        Py_XINCREF(arr);
         m_data = data;
         m_shape = shape;
         m_strides = strides;
