Get rid of most of the cython C code build warnings.

Author: abergeron

pygpu/gpuarray.pxd

@@ -16,7 +16,7 @@ cdef extern from "numpy/arrayobject.h":
 cdef object PyArray_Empty(int a, np.npy_intp *b, np.dtype c, int d)
 
 cdef extern from "Python.h":
-    int PySlice_GetIndicesEx(slice_object slice, Py_ssize_t length,
+    int PySlice_GetIndicesEx(object slice, Py_ssize_t length,
                              Py_ssize_t *start, Py_ssize_t *stop,
                              Py_ssize_t *step,
                              Py_ssize_t *slicelength) except -1

pygpu/gpuarray.pyx

@@ -235,7 +235,7 @@ cdef int strides_ok(GpuArray a, strides):
                 return 0
             upper += max_axis_offset
         else:
-            if lower < -max_axis_offset:
+            if lower < <size_t>(-max_axis_offset):
                 return 0
             lower += max_axis_offset
     return (upper + itemsize) <= size
@@ -874,7 +874,7 @@ def from_gpudata(size_t data, offset, dtype, shape, GpuContext context=None,
         free(cdims)
         free(cstrides)
 
-def array(proto, dtype=None, copy=True, order=None, int ndmin=0,
+def array(proto, dtype=None, copy=True, order=None, unsigned int ndmin=0,
           GpuContext context=None, cls=None):
     """
     array(obj, dtype='float64', copy=True, order=None, ndmin=0, context=None, cls=None)
@@ -890,7 +890,7 @@ def array(proto, dtype=None, copy=True, order=None, int ndmin=0,
     :param order: memory layout of the result
     :type order: string
     :param ndmin: minimum number of result dimensions
-    :type ndmin: int
+    :type ndmin: unsigned int
     :param context: allocation context
     :type context: GpuContext
     :param cls: result class (must inherit from GpuArray)
@@ -1384,21 +1384,24 @@ cdef GpuArray pygpu_reshape(GpuArray a, unsigned int nd, const size_t *newdims,
     if compute_axis < 0:
         array_reshape(res, a, nd, newdims, ord, nocopy)
         return res
-    if compute_axis >= nd:
+    cdef unsigned int caxis = <unsigned int>compute_axis
+    if caxis >= nd:
         raise ValueError("You wanted us to compute the shape of a dimensions that don't exist")
 
     cdef size_t *cdims
     cdef size_t tot = 1
+    cdef unsigned int i
     for i in range(nd):
-        if i != compute_axis:
+        if i != caxis:
             tot *= newdims[i]
     cdims = <size_t *>calloc(nd, sizeof(size_t))
     if cdims == NULL:
         raise MemoryError, "could not allocate cdims"
 
+    cdef size_t d
     for i in range(nd):
         d = newdims[i]
-        if i == compute_axis:
+        if i == caxis:
             d = a.size // tot
 
             if d * tot != a.size:
@@ -1537,7 +1540,7 @@ cdef class GpuArray:
             k = PyNumber_Index(key)
             if k < 0:
                 k += self.ga.dimensions[i]
-            if k < 0 or k >= self.ga.dimensions[i]:
+            if k < 0 or (<size_t>k) >= self.ga.dimensions[i]:
                 raise IndexError, "index %d out of bounds" % (i,)
             start[0] = k
             step[0] = 0
@@ -1546,9 +1549,7 @@ cdef class GpuArray:
             pass
 
         if isinstance(key, slice):
-            # C compiler complains about argument 1 (key) because it's
-            # declared as a PyObject.  But we know it's a slice so it's ok.
-            PySlice_GetIndicesEx(<slice_object>key, self.ga.dimensions[i],
+            PySlice_GetIndicesEx(key, self.ga.dimensions[i],
                                  start, stop, step, &dummy)
             if stop[0] < start[0] and step[0] > 0:
                 stop[0] = start[0]

setup.py

@@ -97,32 +97,41 @@ def __init__(self, *args, **kwargs):
 fullversion = "%s"
 """ % (MAJOR, MINOR, PATCH, SUFFIX, FULLVERSION))
 
+ea = []
+if sys.platform in ('darwin', 'linux'):
+    # Silence unused stuff warnings
+    ea = ["-Wno-unused-variable", "-Wno-unused-function"]
+
 exts = [Extension('pygpu.gpuarray',
                   sources=['pygpu/gpuarray.pyx'],
                   include_dirs=include_dirs,
                   libraries=['gpuarray'],
                   library_dirs=library_dirs,
+                  extra_compile_args=ea,
                   define_macros=[('GPUARRAY_SHARED', None)]
                   ),
         Extension('pygpu.blas',
                   sources=['pygpu/blas.pyx'],
                   include_dirs=include_dirs,
                   libraries=['gpuarray'],
                   library_dirs=library_dirs,
+                  extra_compile_args=ea,
                   define_macros=[('GPUARRAY_SHARED', None)]
                   ),
         Extension('pygpu._elemwise',
                   sources=['pygpu/_elemwise.pyx'],
                   include_dirs=include_dirs,
                   libraries=['gpuarray'],
                   library_dirs=library_dirs,
+                  extra_compile_args=ea,
                   define_macros=[('GPUARRAY_SHARED', None)]
                   ),
         Extension('pygpu.collectives',
                   sources=['pygpu/collectives.pyx'],
                   include_dirs=include_dirs,
                   libraries=['gpuarray'],
                   library_dirs=library_dirs,
+                  extra_compile_args=ea,
                   define_macros=[('GPUARRAY_SHARED', None)]
                   )]