[gstreamer-bugs] [Bug 440389] [API 0.11] Add GST_LEVEL_FIXME

[GStreamer (bugzilla.gnome.org)]

If you have any questions why you received this email, please see the text at
the end of this email. Replies to this email are NOT read, please see the text
at the end of this email. You can add comments to this bug at:
  http://bugzilla.gnome.org/show_bug.cgi?id=440389

  GStreamer | gstreamer (core) | Ver: HEAD CVS




------- Comment #1 from David Schleef  2007-05-22 18:15 UTC -------
If a decoder properly handles a broken file, I don't think it should be
spitting out GST_ERROR()s.  It can either do a GST_ELEMENT_ERROR() and bail
out, or fix up the output and send a discontinuity.  It should probably also
send a message indicating that the stream is broken.

As for GST_FIXME, I think the label "FIXME" is independent of the severity of
the issue, so it shouldn't be a specific severity.  For example, the message
"This code is not yet complete and could cause segfaults" would be a GST_ERROR,
but "Somebody should spend a week sometime and fix this corner case that is
really mostly correct right now anyway" is not.

Also, having a FIXME is a natural occurrence in code.  I wouldn't want to spew
crap to the terminal all the time.

Perhaps a FIXME in the message string would be a better start.


-- 
See http://bugzilla.gnome.org/page.cgi?id=email.html for more info about why you received
this email, why you can't respond via email, how to stop receiving
emails (or reduce the number you receive), and how to contact someone
if you are having problems with the system.

You can add comments to this bug at http://bugzilla.gnome.org/show_bug.cgi?id=440389.