[Bug 270] New: Extended Matroska demuxer

View: New views
5 Messages — Rating Filter:   Alert me  

[Bug 270] New: Extended Matroska demuxer

by Bugzilla from bugzilla-daemon@xine-project.org :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

http://bugs.xine-project.org/show_bug.cgi?id=270

           Summary: Extended Matroska demuxer
           Product: xine-lib
           Version: Mercurial 1.1
          Platform: Other
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: enhancement
          Priority: P3
         Component: Plugins / Demuxer
        AssignedTo: xine-bugs@...
        ReportedBy: gtdev@...


Created an attachment (id=154)
 --> (http://bugs.xine-project.org/attachment.cgi?id=154)
Patch against 9805:3cdfbeb9ef2f for extended Matroska functionality

Hi,

I did some work on the Matroska demultiplexer. Testing time was rather limited,
but it manages to play my movies.

The attached patch (relative to 9805:3cdfbeb9ef2f) adds the following
functionality:

* Read segment title and uses that for display in a UI
There is an issue when the file does not specify a segment title. It will then
fall back to a generic "(No title)", since I could not find a way to retrieve
the file name the player shows.

* More implementation files
Added:
 - demux_matroska.h
 - demux_matroska_chapters.h

This breaks the OO-ish C visibility a bit, since there need to be public (i.e.
non-static) interfaces between the units.

* Chapter Handling
I did a rough initial implementation of Matroska's "editions" system. The
demuxer will parse all editions from the header, and for each edition the top
level of chapters. This is not quite the full spec as Matroska intends, but it
should work fine as long as there is only a single edition and all
editions/chapters only reference only one (the first and only) segment in the
stream, and are supposed to apply to all tracks therein.

When the stream has chapters, the demuxer will now handle skip events from the
player to jump between chapters.


--
Configure bugmail: http://bugs.xine-project.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
You are watching all bug changes.


------------------------------------------------------------------------------
Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day
trial. Simplify your report design, integration and deployment - and focus on
what you do best, core application coding. Discover what's new with
Crystal Reports now.  http://p.sf.net/sfu/bobj-july
_______________________________________________
xine-bugs mailing list
xine-bugs@...
https://lists.sourceforge.net/lists/listinfo/xine-bugs

[Bug 270] Extended Matroska demuxer

by Bugzilla from bugzilla-daemon@xine-project.org :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

http://bugs.xine-project.org/show_bug.cgi?id=270





--- Comment #1 from Nicos Gollan <gtdev@...>  2009-09-17 13:26:44 UTC ---
Created an attachment (id=155)
 --> (http://bugs.xine-project.org/attachment.cgi?id=155)
Additional patch to fix a segfault on cleanup

The first patch will segfault when opening on an invalid stream. This should be
fixed by the small patch.

Details:
I added an event queue. If the stream is not valid EBML, and plugins are chosen
based on content, the error handling code would try to access this->event_queue
at a time when "this" is NULL.


--
Configure bugmail: http://bugs.xine-project.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
You are watching all bug changes.


------------------------------------------------------------------------------
Come build with us! The BlackBerry® Developer Conference in SF, CA
is the only developer event you need to attend this year. Jumpstart your
developing skills, take BlackBerry mobile applications to market and stay
ahead of the curve. Join us from November 9-12, 2009. Register now!
http://p.sf.net/sfu/devconf
_______________________________________________
xine-bugs mailing list
xine-bugs@...
https://lists.sourceforge.net/lists/listinfo/xine-bugs

[Bug 270] Extended Matroska demuxer

by Bugzilla from bugzilla-daemon@xine-project.org :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

http://bugs.xine-project.org/show_bug.cgi?id=270


Nicos Gollan <gtdev@...> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #155 is|0                           |1
           obsolete|                            |
 Attachment #154 is|0                           |1
           obsolete|                            |




--- Comment #2 from Nicos Gollan <gtdev@...>  2009-09-17 17:48:45 UTC ---
Created an attachment (id=156)
 --> (http://bugs.xine-project.org/attachment.cgi?id=156)
New complete patch

New diff that contains all previous changes and a fix for a minor memory leak.

Details:
The demuxer left a small amount of memory behind under this->block_data


--
Configure bugmail: http://bugs.xine-project.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
You are watching all bug changes.


------------------------------------------------------------------------------
Come build with us! The BlackBerry® Developer Conference in SF, CA
is the only developer event you need to attend this year. Jumpstart your
developing skills, take BlackBerry mobile applications to market and stay
ahead of the curve. Join us from November 9-12, 2009. Register now!
http://p.sf.net/sfu/devconf
_______________________________________________
xine-bugs mailing list
xine-bugs@...
https://lists.sourceforge.net/lists/listinfo/xine-bugs

[Bug 270] Extended Matroska demuxer

by Bugzilla from bugzilla-daemon@xine-project.org :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

http://bugs.xine-project.org/show_bug.cgi?id=270





--- Comment #3 from Darren Salt <linux@...>  2009-09-28 21:13:23 UTC ---
Added to the patch queue.


--
Configure bugmail: http://bugs.xine-project.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
You are watching all bug changes.


------------------------------------------------------------------------------
Come build with us! The BlackBerry® Developer Conference in SF, CA
is the only developer event you need to attend this year. Jumpstart your
developing skills, take BlackBerry mobile applications to market and stay
ahead of the curve. Join us from November 9-12, 2009. Register now!
http://p.sf.net/sfu/devconf
_______________________________________________
xine-bugs mailing list
xine-bugs@...
https://lists.sourceforge.net/lists/listinfo/xine-bugs

[Bug 270] Extended Matroska demuxer

by Bugzilla from bugzilla-daemon@xine-project.org :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

http://bugs.xine-project.org/show_bug.cgi?id=270


Darren Salt <linux@...> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|0                           |1




--- Comment #4 from Darren Salt <linux@...>  2009-10-14 00:58:39 UTC ---
Committed, but still needs some testing.

--
Configure bugmail: http://bugs.xine-project.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
You are watching all bug changes.


------------------------------------------------------------------------------
Come build with us! The BlackBerry(R) Developer Conference in SF, CA
is the only developer event you need to attend this year. Jumpstart your
developing skills, take BlackBerry mobile applications to market and stay
ahead of the curve. Join us from November 9 - 12, 2009. Register now!
http://p.sf.net/sfu/devconference
_______________________________________________
xine-bugs mailing list
xine-bugs@...
https://lists.sourceforge.net/lists/listinfo/xine-bugs