[Bug 643252] Improve opencv facedetect performance

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=643252
  GStreamer | gst-plugins-bad | git

Stefan Kost (gstreamer, gtkdoc dev) <ensonic> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #192905|none                        |needs-work
             status|                            |

--- Comment #17 from Stefan Kost (gstreamer, gtkdoc dev) <ensonic at sonicpulse.de> 2011-08-01 12:45:41 UTC ---
Review of attachment 192905:
 --> (https://bugzilla.gnome.org/review?bug=643252&attachment=192905)

::: ext/opencv/gstfacedetect.c
@@ +158,1 @@


why those arbitrary resolutions? I don't think we want to force everyone to
such a low resolution. I would just recommend to use it with videoscale and
capsfilter in case of too high cpu load in a example.

@@ +323,1 @@
 }

if you just lock/unlock the mutex in the property setter you can also remove
it. The setters never run from different processes at the same time (if the app
is done right). We should only use mutexes if we need to ensure that the
processing function is getting a coherent set of properties.

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.