opencv: Remove uneeded buffer writability check
authorNicolas Dufresne <nicolas.dufresne@collabora.com>
Fri, 2 Dec 2016 02:13:23 +0000 (21:13 -0500)
committerNicolas Dufresne <nicolas.dufresne@collabora.com>
Fri, 2 Dec 2016 02:15:28 +0000 (21:15 -0500)
When set to in_place, the BaseTransform class will guaranty that buffers
passed to transform_ip() function are writable.

https://bugzilla.gnome.org/show_bug.cgi?id=775378

ext/opencv/gstfacedetect.cpp
ext/opencv/gstmotioncells.cpp

index 3c82ce0..503b9e7 100644 (file)
@@ -590,19 +590,10 @@ gst_face_detect_transform_ip (GstOpencvVideoFilter * base, GstBuffer * buf,
     vector < Rect > mouth;
     vector < Rect > nose;
     vector < Rect > eyes;
-    gboolean do_display = FALSE;
     gboolean post_msg = FALSE;
 
     Mat mtxOrg (cv::cvarrToMat (img));
 
-    if (filter->display) {
-      if (gst_buffer_is_writable (buf)) {
-        do_display = TRUE;
-      } else {
-        GST_LOG_OBJECT (filter, "Buffer is not writable, not drawing faces.");
-      }
-    }
-
     cvCvtColor (img, filter->cvGray, CV_RGB2GRAY);
 
     gst_face_detect_run_detector (filter, filter->cvFaceDetect,
@@ -719,7 +710,7 @@ gst_face_detect_transform_ip (GstOpencvVideoFilter * base, GstBuffer * buf,
         s = NULL;
       }
 
-      if (do_display) {
+      if (filter->display) {
         CvPoint center;
         Size axes;
         gdouble w, h;
index 33f314a..0aedbd8 100644 (file)
@@ -857,7 +857,6 @@ gst_motion_cells_transform_ip (GstOpencvVideoFilter * base, GstBuffer * buf,
     cellscolor motioncellscolor;
     motioncellidx *motioncellsidx;
 
-    buf = gst_buffer_make_writable (buf);
     if (filter->firstframe) {
       setPrevFrame (img, filter->id);
       filter->firstframe = FALSE;