Use Numpy macros for formatting

jenshnielsen · jenshnielsen · commit 0c0b5dfb5816 · 2016-03-07T20:21:29.000Z
Should ensure correct formatting in both 32 and 64 bit
diff --git a/src/_backend_agg_wrapper.cpp b/src/_backend_agg_wrapper.cpp
@@ -449,14 +449,14 @@ PyRendererAgg_draw_gouraud_triangle(PyRendererAgg *self, PyObject *args, PyObjec
 
     if (points.dim(0) != 3 || points.dim(1) != 2) {
         PyErr_Format(PyExc_ValueError,
-                     "points must be a 3x2 array, got %ldx%ld",
+                     "points must be a 3x2 array, got %"NPY_INTP_FMT"x%"NPY_INTP_FMT,
                      points.dim(0), points.dim(1));
         return NULL;
     }
 
     if (colors.dim(0) != 3 || colors.dim(1) != 4) {
         PyErr_Format(PyExc_ValueError,
-                     "colors must be a 3x4 array, got %ldx%ld",
+                     "colors must be a 3x4 array, got %"NPY_INTP_FMT"x%"NPY_INTP_FMT,
                      colors.dim(0), colors.dim(1));
         return NULL;
     }
@@ -490,21 +490,21 @@ PyRendererAgg_draw_gouraud_triangles(PyRendererAgg *self, PyObject *args, PyObje
 
     if (points.size() != 0 && (points.dim(1) != 3 || points.dim(2) != 2)) {
         PyErr_Format(PyExc_ValueError,
-                     "points must be a Nx3x2 array, got %ldx%ldx%ld",
+                     "points must be a Nx3x2 array, got %"NPY_INTP_FMT"x%"NPY_INTP_FMT"x%"NPY_INTP_FMT,
                      points.dim(0), points.dim(1), points.dim(2));
         return NULL;
     }
 
     if (colors.size() != 0 && (colors.dim(1) != 3 || colors.dim(2) != 4)) {
         PyErr_Format(PyExc_ValueError,
-                     "colors must be a Nx3x4 array, got %ldx%ldx%ld",
+                     "colors must be a Nx3x4 array, got %"NPY_INTP_FMT"x%"NPY_INTP_FMT"x%"NPY_INTP_FMT,
                      colors.dim(0), colors.dim(1), colors.dim(2));
         return NULL;
     }
 
     if (points.size() != colors.size()) {
         PyErr_Format(PyExc_ValueError,
-                     "points and colors arrays must be the same length, got %ld and %ld",
+                     "points and colors arrays must be the same length, got %"NPY_INTP_FMT" and %"NPY_INTP_FMT,
                      points.dim(0), colors.dim(0));
         return NULL;
     }
diff --git a/src/_image_wrapper.cpp b/src/_image_wrapper.cpp
@@ -273,7 +273,7 @@ image_resample(PyObject *self, PyObject* args, PyObject *kwargs)
         } else {
             PyErr_Format(
                 PyExc_ValueError,
-                "If 3-dimensional, array must be RGBA.  Got %ld planes.",
+                "If 3-dimensional, array must be RGBA.  Got %" NPY_INTP_FMT " planes.",
                 PyArray_DIM(input_array, 2));
             goto error;
         }
diff --git a/src/_path_wrapper.cpp b/src/_path_wrapper.cpp
@@ -214,7 +214,7 @@ static PyObject *Py_update_path_extents(PyObject *self, PyObject *args, PyObject
 
     if (minpos.dim(0) != 2) {
         PyErr_Format(PyExc_ValueError,
-                     "minpos must be of length 2, got %ld",
+                     "minpos must be of length 2, got %"NPY_INTP_FMT,
                      minpos.dim(0));
         return NULL;
     }
diff --git a/src/py_converters.cpp b/src/py_converters.cpp
@@ -535,7 +535,7 @@ int convert_points(PyObject *obj, void *pointsp)
 
     if (points->dim(1) != 2) {
         PyErr_Format(PyExc_ValueError,
-                     "Points must be Nx2 array, got %ldx%ld",
+                     "Points must be Nx2 array, got %"NPY_INTP_FMT"x%"NPY_INTP_FMT,
                      points->dim(0), points->dim(1));
         return 0;
     }
@@ -559,7 +559,7 @@ int convert_transforms(PyObject *obj, void *transp)
 
     if (trans->dim(1) != 3 || trans->dim(2) != 3) {
         PyErr_Format(PyExc_ValueError,
-                     "Transforms must be Nx3x3 array, got %ldx%ldx%ld",
+                     "Transforms must be Nx3x3 array, got %"NPY_INTP_FMT"x%"NPY_INTP_FMT"x%"NPY_INTP_FMT,
                      trans->dim(0), trans->dim(1), trans->dim(2));
         return 0;
     }
@@ -583,7 +583,7 @@ int convert_bboxes(PyObject *obj, void *bboxp)
 
     if (bbox->dim(1) != 2 || bbox->dim(2) != 2) {
         PyErr_Format(PyExc_ValueError,
-                     "Bbox array must be Nx2x2 array, got %ldx%ldx%ld",
+                     "Bbox array must be Nx2x2 array, got %"NPY_INTP_FMT"x%"NPY_INTP_FMT"x%"NPY_INTP_FMT,
                      bbox->dim(0), bbox->dim(1), bbox->dim(2));
         return 0;
     }
@@ -607,7 +607,7 @@ int convert_colors(PyObject *obj, void *colorsp)
 
     if (colors->dim(1) != 4) {
         PyErr_Format(PyExc_ValueError,
-                     "Colors array must be Nx4 array, got %ldx%ld",
+                     "Colors array must be Nx4 array, got %"NPY_INTP_FMT"x%"NPY_INTP_FMT,
                      colors->dim(0), colors->dim(1));
         return 0;
     }

Original file line number	Diff line number	Diff line change
`@@ -273,7 +273,7 @@ image_resample(PyObject self, PyObject args, PyObject *kwargs)`
`273`	`273`	`} else {`
`274`	`274`	`PyErr_Format(`
`275`	`275`	`PyExc_ValueError,`
`276`		`- "If 3-dimensional, array must be RGBA. Got %ld planes.",`
	`276`	`+ "If 3-dimensional, array must be RGBA. Got %" NPY_INTP_FMT " planes.",`
`277`	`277`	`PyArray_DIM(input_array, 2));`
`278`	`278`	`goto error;`
`279`	`279`	`}`
Original file line number	Diff line number	Diff line change
`@@ -214,7 +214,7 @@ static PyObject Py_update_path_extents(PyObject self, PyObject *args, PyObject`
`214`	`214`
`215`	`215`	`if (minpos.dim(0) != 2) {`
`216`	`216`	`PyErr_Format(PyExc_ValueError,`
`217`		`- "minpos must be of length 2, got %ld",`
	`217`	`+ "minpos must be of length 2, got %"NPY_INTP_FMT,`
`218`	`218`	`minpos.dim(0));`
`219`	`219`	`return NULL;`
`220`	`220`	`}`