[Bug 797338] dshowvideosrc: add device-index, add GstDeviceMonitor, and several fixes

[GStreamer (GNOME Bugzilla)]

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

--- Comment #21 from Joshua M. Doe <oss at nvl.army.mil> ---
Review of attachment 374048:
 --> (https://bugzilla.gnome.org/review?bug=797338&attachment=374048)

I'm not clear on this one. What problems can arise by opening the device in
start() as opposed to the NULL->READY state change? Looking through many source
elements in -good, some seem to open devices in start(), and some during
NULL->READY. Perhaps the documentation on start() needs to be improved, as it
currently states "Subclasses should open resources and prepare to produce
data". If I should move device opening to the NULL->READY state change, what do
I leave in start()?

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.