GNOME Bugzilla – Bug 328336
silence warings which make dvdec / dvdemux unusable
Last modified: 2006-01-27 12:21:09 UTC
Please describe the problem: after speaking with ddennedy, about libdv printing out error to stderr by default, he suggested to add dv_set_error_log (dv_decoder_t *dv, NULL); in the gst plugin to make dv capture as usable as possible. here is a link to the upstream bugtracker: https://sourceforge.net/tracker/?func=detail&atid=104393&aid=1410072&group_id=4393 Steps to reproduce: Actual results: Expected results: Does this happen every time? Other information:
Created attachment 57960 [details] [review] here is a patch adding command to silence warings...
j^: Can you make sure that that symbol is available? I.e. figure out at what version libdv started having that symbol, and then either require that version or (better) add an ifdef to run that line only if the version is new enough. Also add a Changelog entry in your patch please :) p.s. edit your bugzilla profile to put your real name :-)
looking at the changelog in libdv this was introduced in version 0.98 (released 2002-07-30). not sure if checking is needed at all. --snip < libdv/ChangeLog 2002-05-01 Stefan Lucke <Stefan.Lucke1@epost.de> * audio error correction: default error reporting is set to: stderr old_file = dv_set_error_log (de --snip
Created attachment 58017 [details] [review] patch with ChangeLog
2006-01-27 Jan Gerber <j@bootlab.org> Reviewed by: Andy Wingo <wingo@pobox.com> * ext/dv/gstdvdec.c (gst_dvdec_change_state): * ext/dv/gstdvdemux.c (gst_dvdemux_change_state): Call dv_set_error_log (dv_decoder_t *, NULL); after dv_decoder_new to not have warings flooding stderr. this is the suggested way also used in dvgrab and kino. (#328336)