[Bug 652150] New: [lamemp3enc] Lame encoder generates incorrect timestamps

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=652150
  GStreamer | gst-plugins-ugly | git

           Summary: [lamemp3enc] Lame encoder generates incorrect
                    timestamps
    Classification: Platform
           Product: GStreamer
           Version: git
        OS/Version: All
            Status: UNCONFIRMED
          Severity: normal
          Priority: Normal
         Component: gst-plugins-ugly
        AssignedTo: gstreamer-bugs at lists.freedesktop.org
        ReportedBy: matej.knopp at gmail.com
         QAContact: gstreamer-bugs at lists.freedesktop.org
     GNOME version: ---


Created an attachment (id=189507)
 View: https://bugzilla.gnome.org/attachment.cgi?id=189507
 Review: https://bugzilla.gnome.org/review?bug=652150&attachment=189507

Initial patch

The encoder takes timestamp from input buffer and calculates the duration
according to number of samples in input buffer and then sets these on output
buffer.

This is wrong however as Lame output is always (unless flushed) shorter than
the length of input buffer. Lame buffers certain number of samples, those will
be outputted in next iteration, however the timesamp and duration on output
buffer don't reflect this.

Attached a patch fixing this.

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