[gstreamer-bugs] [Bug 370865] New: [basesrc] update segment duration in gst_base_src_update_length

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

  GStreamer | gstreamer (core) | Ver: HEAD CVS

           Summary: [basesrc] update segment duration in
                    gst_base_src_update_length
           Product: GStreamer
           Version: HEAD CVS
          Platform: Other
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: Normal
         Component: gstreamer (core)
        AssignedTo: gstreamer-bugs at lists.sourceforge.net
        ReportedBy: jonathan at kaolin.wh9.net
         QAContact: gstreamer-bugs at lists.sourceforge.net
     GNOME version: Unspecified
   GNOME milestone: Unspecified


While poking around at bug 341078, I noticed that when basesrc calls
bclass->get_size() to update the length of the file, it doesn't store the
result, so from then on, every call to gst_base_src_update_length() for a read
past the initial size of the file will result in a (possibly expensive) call to
bclass->get_size().  It looks like it would be a good idea to update the
segment duration with the new size to avoid future get_size calls.


-- 
Configure bugmail: http://bugzilla.gnome.org/userprefs.cgi?tab=email