Minor C++ improvements

Author: mdboom

src/cntr.c

@@ -717,6 +717,8 @@ slit_cutter (Csite * site, int up, int pass2)
 
     if (up)
     {
+        assert(pass2);
+
         /* upward stroke of slit proceeds up left side of slit until
          * it hits a boundary or a point not between the contour levels
          * -- this never happens on the first pass */
@@ -1406,7 +1408,11 @@ int reorder(double *xpp, double *ypp, short *kpp,
 
     /* Find the subpaths as sets of connected segments. */
 
-    subp = (int *)malloc(nsegs * sizeof(int));
+    if (nsegs) {
+        subp = (int *)malloc(nsegs * sizeof(int));
+    } else {
+        subp = NULL;
+    }
     for (i=0; i<nsegs; i++) subp[i] = -1;
 
     nsp = 0;
@@ -2025,5 +2031,3 @@ init_cntr(void)
         return m;
     #endif
 }
-
-

src/ft2font.cpp

@@ -68,16 +68,13 @@ void FT2Image::resize(long width, long height)
             delete[] m_buffer;
             m_buffer = NULL;
             m_buffer = new unsigned char[numBytes];
+            memset(m_buffer, 0, numBytes);
         }
 
         m_width = (unsigned long)width;
         m_height = (unsigned long)height;
     }
 
-    if (numBytes) {
-        memset(m_buffer, 0, numBytes);
-    }
-
     m_dirty = true;
 }
 

src/numpy_cpp.h

@@ -425,6 +425,7 @@ class array_view : public detail::array_view_accessors<array_view, T, ND>
         PyArrayObject *tmp;
 
         if (arr == NULL || arr == Py_None) {
+            Py_XDECREF(m_arr);
             m_arr = NULL;
             m_data = NULL;
             m_shape = zeros;
@@ -440,6 +441,7 @@ class array_view : public detail::array_view_accessors<array_view, T, ND>
             }
 
             if (PyArray_NDIM(tmp) == 0 || PyArray_DIM(tmp, 0) == 0) {
+                Py_XDECREF(m_arr);
                 m_arr = NULL;
                 m_data = NULL;
                 m_shape = zeros;

src/py_converters.cpp

@@ -493,16 +493,14 @@ int convert_offset_position(PyObject *obj, void *offsetp)
     e_offset_position *offset = (e_offset_position *)offsetp;
     const char *names[] = {"data", NULL};
     int values[] = {OFFSET_POSITION_DATA};
-    int result;
-
-    *offset = OFFSET_POSITION_FIGURE;
+    int result = OFFSET_POSITION_FIGURE;
 
-    if (convert_string_enum(obj, "offset_position", names, values, &result)) {
-        *offset = (e_offset_position)result;
-    } else {
+    if (!convert_string_enum(obj, "offset_position", names, values, &result)) {
         PyErr_Clear();
     }
 
+    *offset = result;
+
     return 1;
 }