Merge pull request #3 from ties/fix/second_call_flow

apply earlier fixes to second control flow

xbob/flandmark/__init__.py

@@ -120,7 +120,7 @@ class Localizer(_flandmark.Localizer):
       return super(Localizer, self).__call__(gray, b_y, b_x, b_height, b_width)
 
     elif image.ndim == 2:
-      return super(Localizer, self).__call__(gray, b_y, b_x, b_height, b_width)
+      return super(Localizer, self).__call__(image, b_y, b_x, b_height, b_width)
 
     else:
       raise TypeError, "Localizer accepts images as numpy.ndarray objects with either 2 or 3 dimensions"

xbob/flandmark/ext/ext.cpp

@@ -139,17 +139,20 @@ class Localizer {
       ipl_image->imageData = (char*)input.bz<uint8_t,2>().data();
 
       int bbox[4] = {b_x, b_y, b_x + b_width, b_y + b_height};
+      int flandmark_result;
       {
         bob::python::no_gil unlock;
-        flandmark_detect(ipl_image.get(), bbox, m_flandmark.get(),
+        flandmark_result = flandmark_detect(ipl_image.get(), bbox, m_flandmark.get(),
             m_landmarks.get());
       }
       
       list lmlist; ///< landmark list
 
+      if (flandmark_result == NO_ERR) {
         for (int i = 0; i < (2*m_flandmark->data.options.M); i += 2) {
           lmlist.append(make_tuple(m_landmarks[i], m_landmarks[i+1]));
         }
+      }
 
       return object(lmlist);
     }