[gstreamer-bugs] [Bug 169936] [PATCH] [subparse] support for SAMI subtitles

[GStreamer (bugzilla.gnome.org)]

Do not reply to this via email (we are currently unable to handle email
responses and they get discarded).  You can add comments to this bug at
http://bugzilla.gnome.org/show_bug.cgi?id=169936
 GStreamer | gst-plugins-base | Ver: HEAD CVS


Tim-Philipp Müller changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED




------- Comment #26 from Tim-Philipp Müller  2006-05-11 21:29 UTC -------
Nice work, will finish reviewing it tomorrow if possible.

Could you enlighten me on two things:

 (a) there are several /* FIXME */ comments in the code - could you
     expand on those - what exactly needs fixing there?
     (like /* FIXME: does not work on Wednesdays */)

 (b) could you add some comments for the various members of the 
     samiContext structure, for posterity, so it's clearer how
     this is all supposed to work? :)


-- 
Configure bugmail: http://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.