[gst-devel] Re: [gst-cvs] thomasvs gstreamer: gstreamer/ gstreamer/gst/

[Ronald S. Bultje]

Hi,

On Mon, 2005-02-14 at 11:15, Thomas Vander Stichele wrote:
> > On Fri, 2005-02-11 at 14:04, Thomas Vander Stichele wrote:
> > >           remove unused and unneeded error to combat inflation
> > [..]
> > > -  TABLE (t, STREAM, INVALID_DATA,
> > > -      N_("The stream contained invalid data and cannot be processed."));
[..]
> The point of these error domains is to not have a different one for each
> possible error.  This one doesn't need to be added because it fits under
> the other.  Also, nothing in gstreamer or gst-plugins (in the threaded
> branch) uses it.  So I don't even see why it was added in the first
> place.

(avidemux uses it... See below.)

> Anyway, like I said a long time ago - if people really feel one needs to
> be added, I want it discussed, precisely because I don't want an
> inflation of error enums, *especially* not in a development branch.

Right, that's a good concern. I'd like it added to error out on
corrupted data streams (e.g. download errors or so). Which current one
do you suggest? I may have not looked correctly when I added it...

Ronald

-- 
Ronald S. Bultje <rbultje at ronald.bitfreak.net>