[GEM-dev] Gem patches to fix DV4L bugs and crashes

IOhannes m zmoelnig zmoelnig at iem.at
Mon Feb 19 12:53:03 CET 2007


hi.

i have applied the change to videoDV4L, but i think the
pix_vieoNEW.cpp-issue should be handled differently:

Ivica Ico Bukvic wrote:

> +NB: It is very likely that it may be necessary for this method to exist for other types of video devices, but since I was unable to test them, I've put it under the most restrictive ifdef available.
> +
> +Ico Bukvic ico at vt.edu 2-18-07
> +*/
> +#ifdef HAVE_LIBDV
> +  if (m_videoHandle)m_videoHandle->stopTransfer();
> +#endif /* DV4L */
>  }
>  

this is weird and does not make full sense to me. e.g. if you have
compiled Gem with v4l AND libdv support (2 drivers), then the
stopTransfer() of the currently used handle will be called on
destruction of the [pix_video] object. (this could as well be the
V4L-handle; so the define does not exactly what it suggests that it
might do)

i have replaced your suggestion by calling the destructor of all
existing handles on destruction of pix_videoNEW.

please check whether this still fixes your issues.


fgmad.r
IOhannes




More information about the GEM-dev mailing list