[gstreamer-bugs] [Bug 601576] qtmux enhancements (moov recovery, mem usage and edit lists)

GStreamer (bugzilla.gnome.org) bugzilla at gnome.org
Wed Nov 11 09:52:32 PST 2009


https://bugzilla.gnome.org/show_bug.cgi?id=601576
  GStreamer | gst-plugins-bad | 0.10.14

Tim-Philipp Müller <t.i.m> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEEDINFO
                 CC|                            |t.i.m at zen.co.uk

--- Comment #1 from Tim-Philipp Müller <t.i.m at zen.co.uk> 2009-11-11 17:52:30 UTC ---
Hrm, cool stuff (I think?), but this patch is really quite large. Do you think
you could

 - submit a patch that does not include generated files
   (Makefile.in in particular), preferably against git,
   especially for such a large patch

 - open separate bugs for separate issues (the edit list
   support does not depend on the other stuff, does it?)

 - not include the faac fix (please just ping the existing
   bug if we forgot about it)

? This would make reviewing and discussing this *much* easier.

I must say the xml serialisation/deserialisation looks quite painful. Is this
really the easiest/best way to do this? Not only does it seem awkward and
unnecessarily verbose to me, but I'm also not very keen on seeing an external
dependency added just for this feature (others might disagree of course).

-- 
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