After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 602508 - qtdemux: Parse stbl atom per sample instead of all at once
qtdemux: Parse stbl atom per sample instead of all at once
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-good
git master
Other Linux
: Normal normal
: 0.10.18
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2009-11-20 16:04 UTC by Robert Swain
Modified: 2009-11-23 16:32 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Robert Swain 2009-11-20 16:04:27 UTC
I have been working to make the qtdemux element parse the stbl atom per sample to improve start-up times when playing mov/mp4/etc.

In master, prior to these changes, the entire atom is parsed during pre-rolling. My changes make the code parse the 0th sample during pre-rolling and then any others as required to decode samples. Note that if a seek occurs, the entire remainder of the atom is parsed.

I have published the changes in the working-qtdemux branch in my personal -good repository at git://people.freedesktop.org/~robswain/gst-plugins-good

Review and comments are very welcome. The changes have been tested quite a lot, and while I'm not really aware of any outstanding issues, maybe some still remain.
Comment 1 Robert Swain 2009-11-23 16:02:27 UTC
Tim and Wim were happy with the state of these patches and Wim requested that they be pushed to master, so I have just done that. Closing...
Comment 2 Jan Schmidt 2009-11-23 16:32:19 UTC
Don't forget to set the target to the next release of the module when closing bugs, so the release scripts can include it in the release notes.