Support for EDL (Edit Decision List chapter lists)?

DrDDT · 2007-08-01 21:58

elupus Wrote:actually mplayer normally supports this (edl lists), however it's currently disable din our mplayer.dll build. will make sure that is enabled in the upcomming build thou.

How can I check if it's in now, and if it's in the DLL, how can I use it?
Do I have to submit an official feature request on sourceforge for this?

**elupus** · 2007-08-02 15:50

it isn't in. i had forgot about it

DrDDT · 2007-08-02 17:58

elupus Wrote:it isn't in. i had forgot about it

I'd still would like to have this feature, so please add it.

**spiff** · 2007-08-02 18:34

patch welcome.

if not sit tight and maybe elupus adds it when he feels like it....

DrDDT · 2007-08-03 19:47

spiff Wrote:patch welcome.

if not sit tight and maybe elupus adds it when he feels like it....

I thought the code was already in mplayer, just not enabled for XBMC?

**elupus** · 2007-08-04 12:17

it's in the new mplayer.dll in svn now.

DrDDT · 2007-08-04 23:21

elupus Wrote:it's in the new mplayer.dll in svn now.

It is!

Code:

2007-08-03  9787  updated: mplayer.dll ( enabled edl list, fixed asv/wmv bug with streams missing extended properties )

Thanks!

DrDDT · 2007-08-06 20:02

kraqh3d Wrote:No, thats just one way it could be implemented later as a full-fledged feature:
1) check for existance of filename.edl in the same folder as filename
2) if it exists, cache it as "z:\edl.edl"
3) when mplayer starts up, pass it "edl=z:\edl.edl" under the covers

In the meantime, you'll have to manually copy the edl file to the harddrive (any drive will do), and make sure your filename.ext.conf file (so you dont have to mess with the mpalyer/conf file) references the edl file on the hard drive with a fully qualified path. And assuming you name them all uniquely, and put them either on the C or E drive, you dont really need to delete them afterwards.

you should be able to use a filename.ext.conf file which contains:
edl=e:\edls\movie.edl

Say I would want to implement this, and use the subtite handling code as an example, where can I find this code? Is it in xbmc somewhere, or is this mplayer functionality?

**spiff** · 2007-08-07 00:27

i guess the most convenient place to fit it in would be in CUtil::CacheSubtitles() (Util.cpp) plus an additional check for a edl where the mplayer command line options are setup (MPlayer.cpp)

this would be all in xbmc, no mplayer changes required

DrDDT · 2007-08-07 00:30

spiff Wrote:i guess the most convenient place to fit it in would be in CUtil::CacheSubtitles() (Util.cpp) plus an additional check for a edl where the mplayer command line options are setup (MPlayer.cpp)

this would be all in xbmc, no mplayer changes required

Thanks, I'll give it a try.

DrDDT Senior Member Posts: 186 Joined: Apr 2006 Reputation: 0	2007-08-01 21:58 Post: #31 elupus Wrote:actually mplayer normally supports this (edl lists), however it's currently disable din our mplayer.dll build. will make sure that is enabled in the upcomming build thou. How can I check if it's in now, and if it's in the DLL, how can I use it? Do I have to submit an official feature request on sourceforge for this?
	find quote

elupus Team-XBMC Developer Posts: 3,764 Joined: Mar 2004 Reputation: 2	2007-08-02 15:50 Post: #32 it isn't in. i had forgot about it Always read the XBMC online-manual, FAQ and search the forum before posting. Do not e-mail XBMC-Team members directly asking for support. Read/follow the forum rules. For troubleshooting and bug reporting please make sure you read this first.
	find quote

DrDDT Senior Member Posts: 186 Joined: Apr 2006 Reputation: 0	2007-08-02 17:58 Post: #33 elupus Wrote:it isn't in. i had forgot about it I'd still would like to have this feature, so please add it.
	find quote

spiff Grumpy Bastard Developer Posts: 12,180 Joined: Nov 2003 Reputation: 82	2007-08-02 18:34 Post: #34 patch welcome. if not sit tight and maybe elupus adds it when he feels like it.... Always read the XBMC online-manual, FAQ and search the forum before posting. Do not e-mail XBMC-Team members directly asking for support. Read/follow the forum rules. For troubleshooting and bug reporting please make sure you read this first.
	find quote

DrDDT Senior Member Posts: 186 Joined: Apr 2006 Reputation: 0	2007-08-03 19:47 Post: #35 spiff Wrote:patch welcome. if not sit tight and maybe elupus adds it when he feels like it.... I thought the code was already in mplayer, just not enabled for XBMC?
	find quote

elupus Team-XBMC Developer Posts: 3,764 Joined: Mar 2004 Reputation: 2	2007-08-04 12:17 Post: #36 it's in the new mplayer.dll in svn now. Always read the XBMC online-manual, FAQ and search the forum before posting. Do not e-mail XBMC-Team members directly asking for support. Read/follow the forum rules. For troubleshooting and bug reporting please make sure you read this first.
	find quote

DrDDT Senior Member Posts: 186 Joined: Apr 2006 Reputation: 0	2007-08-04 23:21 Post: #37 elupus Wrote:it's in the new mplayer.dll in svn now. It is! Code: `2007-08-03 9787 updated: mplayer.dll ( enabled edl list, fixed asv/wmv bug with streams missing extended properties )` Thanks!
	find quote

DrDDT Senior Member Posts: 186 Joined: Apr 2006 Reputation: 0	2007-08-07 00:30 Post: #40 spiff Wrote:i guess the most convenient place to fit it in would be in CUtil::CacheSubtitles() (Util.cpp) plus an additional check for a edl where the mplayer command line options are setup (MPlayer.cpp) this would be all in xbmc, no mplayer changes required Thanks, I'll give it a try.
	find quote