[Bug 703283] id3: gst_date_time_new: assertion `(month > 0 && month <= 12) || month == -1' failed with malformed TDAT frames

GStreamer (bugzilla.gnome.org) bugzilla at gnome.org
Mon Jul 1 02:37:14 PDT 2013


https://bugzilla.gnome.org/show_bug.cgi?id=703283
  GStreamer | gst-plugins-base | 1.0.7

--- Comment #9 from Marcin Lewandowski <marcin at saepia.net> 2013-07-01 09:37:10 UTC ---
Hey Tim, I've added test cases.

I disagree that it should extract partial information (only month in this case,
as GstDateTime does not support only days). Assumption that month is valid if
syntax of whole frame is invalid is quite optimistic and output can be
confusing to the user. IMO GStreamer should be rather strict by default. 

Comparing this to the web browser world, IMHO GStreamer shouldn't act as IE
which was trying to guess underskilled author's intention and introduced
several rendering modes which in fact were causing even more trouble. It should
be rather equivalent of Firefox enforcing standards.

Please confirm, if you still think I should add this functionality to the
patch.

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.


More information about the gstreamer-bugs mailing list