[Bug 629244] [opencv] Add motion detection element

[GStreamer (bugzilla.gnome.org)]

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

--- Comment #129 from Nicola <lists at svrinformatica.it> 2011-07-17 10:20:28 UTC ---
Hi,

I fixed the mutex things too, before post the patch I would like to ask for
some advices:

1) actually we lock in gst_motion_cells_chain, so every time we process a frame
and when we set a property (gst_motion_cells_set_property), this allow to
dynamically change a property in a safe way, however I think not all will use
the plugin this way and a lock for every frame could be expensive, I would like
to make a property to enable this, I think the default should be disabled.
Robert, Stefan, what do you think about?
2) Robert setted a mutex in gst_motion_cells_set_property before the "switch
(prop_id)" this lock the pipeline posted above where I set multiple property
using gst-launch, so to solve this I setted the muxex for each property and
this work but is less convenient. Stefan, is there a way to make it work the
Robert way, or a way to protect with a mutex the whole set_property method? I
would like not lock/release every property but the whole method,

thanks
Nicola

-- 
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.