[gstreamer-bugs] [Bug 334083] New: jpegdec buffer duration (s)

[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=334083
 GStreamer | gst-plugins-good | Ver: 0.10.2

           Summary: jpegdec buffer duration (s)
           Product: GStreamer
           Version: 0.10.2
          Platform: Other
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: Normal
         Component: gst-plugins-good
        AssignedTo: gstreamer-bugs at lists.sourceforge.net
        ReportedBy: mdk at mdk.org.pl
         QAContact: gstreamer-bugs at lists.sourceforge.net
     GNOME version: Unspecified
   GNOME milestone: Unspecified


There is something wrong with the GST_BUFFER_DURATION of the buffers generated
by jpegdec. It seems to boil down to:

gstjpegdec.c , line 823+

...
GST_BUFFER_TIMESTAMP (outbuf) = dec->next_ts;

if (dec->packetized && dec->framerate_numerator != 0) {
  GstClockTime next_ts;

  dec->frames_decoded++;
  next_ts = dec->frames_decoded * GST_SECOND *
      dec->framerate_denominator / dec->framerate_numerator;

  duration = next_ts - dec->next_ts;
  dec->next_ts = next_ts;
} else {
  duration = GST_CLOCK_TIME_NONE;
  dec->next_ts = GST_CLOCK_TIME_NONE;
}
GST_BUFFER_DURATION (outbuf) = duration;
...

Basically 'next_ts - dec->next_ts' is sometimes negative. I bumped on that with
MJPEG AVI's after seek. Reported on irc too.


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