[gst-devel] [gst-cvs] thomasvs gst-ffmpeg: gst-ffmpeg/ gst-ffmpeg/ext/ffmpeg/

[Thomas Vander Stichele]

Hi,

On Sat, 2007-02-10 at 11:10 +0100, Edward Hervey wrote:
> Hi,
> 
>   The reason this warning is there is for one specific case:
>    * When we get a ffmpeg codecid that we haven't identified in
> gst-ffmpeg, whether that codec is wrapped or not !
>    * Having unknown codecid is extremely bad, but since we are at the
> register level we cannot emit a error/warning message on the bus
> (there is no element, and therefore bus on which we can send this
> message)
>    * AFAIK, all codecid of the ffmpeg snapshot are identified. The
> only case where you would see this is if you compile gst-ffmpeg with
> an external ffmpeg which we DO NOT support.
> 
>   Thomas, if you are seeing this warning this means you are using a
> non-supported gst-ffmpeg. Please revert that commit since it is needed
> to identify non-identified codecids in the *official* gst-ffmpeg.

I understand all of these reasons.  At the end of the line we should
STILL not be outputting all this crap to console by default, ESPECIALLY
if the user has *no idea at all* what to do with these messages.  It
took quite some time myself to look into where the heck these things
were coming from.  How is an end user supposed to be able to tell you ?

Make them WARNING or whatever, but do not have them output on the
console.

Thomas

>    Edward
> 
> On 2/9/07, Thomas Vander Stichele <thomasvs at kemper.freedesktop.org> wrote:
> > CVS Root:       /cvs/gstreamer
> > Module:         gst-ffmpeg
> > Changes by:     thomasvs
> > Date:           Fri Feb 09 2007  19:58:31 UTC
> >
> > Log message:
> >         * ext/ffmpeg/gstffmpegdec.c: (gst_ffmpegdec_register):
> >           My console is not your TODO list.
> >           If it were your TODO list, you would have DONE it by now.
> >
> > Modified files:
> >     .               : ChangeLog
> >     ext/ffmpeg      : gstffmpegdec.c
> >
> > Links:
> > http://freedesktop.org/cgi-bin/viewcvs.cgi/gstreamer/gst-ffmpeg/ChangeLog.diff?r1=1.373&r2=1.374
> > http://freedesktop.org/cgi-bin/viewcvs.cgi/gstreamer/gst-ffmpeg/ext/ffmpeg/gstffmpegdec.c.diff?r1=1.169&r2=1.170
> >
> > -------------------------------------------------------------------------
> > Using Tomcat but need to do more? Need to support web services, security?
> > Get stuff done quickly with pre-integrated technology to make your job easier.
> > Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
> > http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
> > _______________________________________________
> > gstreamer-cvs mailing list
> > gstreamer-cvs at lists.sourceforge.net
> > https://lists.sourceforge.net/lists/listinfo/gstreamer-cvs
> >
> 
>