[gstreamer-bugs] [Bug 504081] [matroskamux] support for live sources (calculate duration, accept mjpeg without framerate)

[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=504081

  GStreamer | gst-plugins-good | Ver: HEAD CVS

Tim-Philipp Müller changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #101130|none                        |committed
               Flag|                            |
         AssignedTo|t.i.m at zen.co.uk             |gstreamer-
                   |                            |bugs at lists.sourceforge.net
             Status|ASSIGNED                    |RESOLVED
         Resolution|                            |FIXED
            Summary|Support for live sources in |[matroskamux] support for
                   |matroskamux.                |live sources (calculate
                   |                            |duration, accept mjpeg
                   |                            |without framerate)
   Target Milestone|HEAD                        |0.10.7




------- Comment #2 from Tim-Philipp Müller  2008-01-08 19:59 UTC -------
Committed with a bunch of gratuitious changes and small clean-ups:

 2008-01-08  Tim-Philipp Müller  <tim at centricular dot net>

        Based on patch by: Wouter Cloetens  <wouter at mind be>

        * gst/matroska/matroska-mux.c: (gst_matroska_mux_video_pad_setcaps),
          (gst_matroska_mux_request_new_pad), (gst_matroska_mux_release_pad),
          (gst_matroska_mux_finish), (gst_matroska_mux_collected):
        * gst/matroska/matroska-mux.h:
          Keep track of first and last timestamps for each incoming stream,
          so we can calculate the total duration for live sources and other
          input where we can't query the duration from the start or where
          there's no constant framerate from which we can deduce the
          duration; also use calculated/observed duration if it is bigger
          than the previously queried duration. Furthermore, use
          gst_pad_query_peer_duration() and take into account that it may
          return TRUE but still a duration of CLOCK_TIME_NONE, which easily
          screws up comparisons when using unsigned integers. Fixes #504081.

Please test and let me know if I screwed up something.


-- 
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=504081.